Skip to content
This repository was archived by the owner on Nov 14, 2024. It is now read-only.

Gulp Fix#15

Open
erickocrs wants to merge 3 commits intotonik:masterfrom
erickocrs:gulp_fix
Open

Gulp Fix#15
erickocrs wants to merge 3 commits intotonik:masterfrom
erickocrs:gulp_fix

Conversation

@erickocrs
Copy link
Copy Markdown

Fix Gulp/Node Updates

Comment thread build/build.js
gulp.task('sass:clean', require('./tasks/sass/clean'))
gulp.task('sass:lint', require('./tasks/sass/lint'))
gulp.task('sass:build', require('./tasks/sass/build'))
gulp.task('sass:clean', (cb) => { require('./tasks/sass/clean') (cb())})
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You could simplify it to:
cb => require('./tasks/sass/clean') (cb())

Comment thread build/build.js

gulp.watch('../src/**/*.scss', gulp.series(['sass']))
.on('error', message.error('WATCH: Sass'))
.on("change", gulp.series(reload))
Copy link
Copy Markdown

@eaitbrahim eaitbrahim Oct 26, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please be consistent with the use of quotes, if you're using a single quote then just keep using it...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants