Gulpfile.js guardano le migliori pratiche

Question

ecco la mia attuale lista per i miei gulpfile.js

// Gulp watchlist 
gulp.task('watch', ['browserSync', 'sass'], function(){ 
    gulp.watch('app/scss/**/*.scss', ['sass']); 
    gulp.watch('app/*.html').on('change', browserSync.reload); 
    gulp.watch('app/js/**/*.js').on('change', browserSync.reload); 
    //add more watchers here 
}); 

e questo funziona.

ma un tutorial Seguo ha qualcosa di leggermente diverso:

gulp.task('watch', ['browserSync', 'sass'], function(){ 
    gulp.watch('app/scss/**/*.scss', ['sass']); 
    // Reloads the browser whenever HTML or JS files change 
    gulp.watch('app/*.html', browserSync.reload); 
    gulp.watch('app/js/**/*.js', browserSync.reload); 
}); 

ho bisogno del .on ('cambiamento', browserSync.reload)? Funziona; Sono solo curioso di sapere se quello che sto facendo non è una buona pratica. Grazie!

Answer 1

No, non è necessario aggiungere personalizzato on('change'). Dovresti passare il gestore di eventi durante l'inizializzazione del watcher come il tutorial. Altrimenti, è come dire a Gulp di non fare nulla in cambio di default, quindi aggiungere successivamente il listener.

Un altro vantaggio è che si può passare in più gestori di eventi in una sola volta come un array se i gestori sono compiti gulp stessi gulp.watch(..., ['task1', 'task2']) invece di concatenamento on('change', task1Function).on('change', task2Function)

Answer 2

Gulp watch restituisce sempre un EventEmitter che emette eventi di modifica, quindi la chiamata gulp.watch('app/*.html').on('change', browserSync.reload) può essere scritta come:

var watcher = gulp.watch('app/*.html'); 
watcher.on('change', browserSync.reload); 

watch funzione può essere richiamata con le seguenti params: gulp.watch(glob [, opts], tasks) o gulp.watch(glob [, opts, cb]).

Quindi è possibile utilizzare entrambe le varianti, ma è preferibile il metodo più breve.