Gulp.js: task based on forEach loop

Question

I have an array of objects that looks like the following.

var bundles = [
  {
    src: 'js/my-component/*.js',
    bundleName: 'my-component.js'
  },
  {
    src: 'js/my-other-component/*.js',
    bundleName: 'my-other-component.js'
  }
]

I want the gulp task to process/concat every entry in the array, but it doesn't seem to work.

gulp.task('bundlejs', function(){
    return bundles.forEach(function(obj){
      return gulp.src(obj.src)
      .pipe(concat(obj.bundleName))
      .pipe(gulp.dest('js/_bundles'))
    });
});

score 13 · Accepted Answer · answered Aug 19 '15 at 04:37

13

You should probably be merging the streams and returning the result, so that the task will complete at the appropriate time:

var es = require('event-stream');

gulp.task('bundlejs', function () {
  return es.merge(bundles.map(function (obj) {
    return gulp.src(obj.src)
      .pipe(concat(obj.bundleName))
      .pipe(gulp.dest('js/_bundles'));
  }));
});

answered Aug 19 '15 at 04:37

Ben

10,056
5
41
42

Thanks Ben, I will try this. – gkiely Aug 19 '15 at 23:34
I marked this as the answer as it highlighted a best practice for structuring the code I had submitted. The issue I had was typo as detailed below. – gkiely Sep 21 '16 at 01:05

score 0 · Answer 2 · answered Aug 17 '15 at 23:54

0

This solution does work, I had the wrong directory name. Doh.

https://github.com/adamayres/gulp-filelog helped me find the problem.

answered Aug 17 '15 at 23:54

gkiely

2,987
1
23
37

Gulp.js: task based on forEach loop

2 Answers2