-
Notifications
You must be signed in to change notification settings - Fork 162
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix vulnerabilities (gulp, gulp-mocha, gulp-mustache, gulp-if, del) #1642
Conversation
@testcafe-build-bot retest |
@testcafe-build-bot retest |
@testcafe-build-bot retest |
}); | ||
|
||
gulp.task('client-scripts-bundle', ['clean'], () => { | ||
gulp.step('client-scripts-bundle', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess we can migrate to gulp 4 using a more simple manner
//Gulp 3
gulp.task('client-scripts', ['client-scripts-bundle'], () => {
});
//Gulp 4
gulp.task('client-scripts', gulp.series(['client-scripts-bundle'], () => {
}));
See https://github.com/gulpjs/gulp/blob/4.0/docs/API.md#gulpseriestasks
Check it and discuss with @AndreyBelym
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After discussing with @AndreyBelym we decide to use gulp-step
.
@testcafe-build-bot retest |
Gulpfile.js
Outdated
@@ -148,12 +138,28 @@ gulp.task('client-scripts-bundle', ['clean'], () => { | |||
.pipe(gulp.dest('./lib/client')); | |||
}); | |||
|
|||
gulp.task('server-scripts', ['clean'], () => { | |||
gulp.step('client-scripts-render', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rename to gulp.step('client-scripts-processing')
Gulpfile.js
Outdated
|
||
// Test | ||
gulp.task('test-server', ['build'], () => { | ||
gulp.step('test-server-run', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gulp.step('mocha')
Gulpfile.js
Outdated
gulp.watch('./src/**', ['build']); | ||
gulp.task('test-server', gulp.series('build', 'test-server-run')); | ||
|
||
gulp.step('test-client-run', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gulp.step('qunit', ()
Gulpfile.js
Outdated
|
||
gulp.task('test-client-travis', ['build'], () => { | ||
gulp.step('test-client-travis-run', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gulp.step('travis-saucelabs-qunit'
Gulpfile.js
Outdated
gulp.task('http-playground', ['set-dev-mode', 'build'], () => { | ||
gulp.task('test-client-travis', gulp.series('build', 'test-client-travis-run')); | ||
|
||
gulp.step('http-playground-run', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gulp.step('http-playground-server'
Gulpfile.js
Outdated
gulp.task('https-playground', ['set-dev-mode', 'build'], () => { | ||
gulp.task('http-playground', gulp.series('set-dev-mode', 'build', 'http-playground-run')); | ||
|
||
gulp.step('https-playground-run', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gulp.step('https-playground-server'
@testcafe-build-bot retest |
…evExpress#1642) * fix vulnerabilities * update publish-please * restore publish-please * gulp-ll-next * update gulp-ll-next * update gulp-ll-next * requested changes
Before:
Current:
Changes:
gulp
,gulp-mocha
,gulp-mustache
,gulp-if
,del
.gulp-step
forgulp
tasksgulp-ll-next
instead ofgulp-ll
npm install --save-dev [email protected]
https://github.com/sindresorhus/del/releases/tag/v2.0.0:
Gulp tool window