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'))
    });
});

Ben · Accepted Answer · 2015-08-19 04:37:35Z

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, 2015 at 4:37

Ben

10.1k5 gold badges43 silver badges42 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

gkiely Over a year ago

Thanks Ben, I will try this.

gkiely Over a year ago

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 · Accepted Answer · 2015-08-17 23:54:26Z

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, 2015 at 23:54

gkiely

3,0071 gold badge26 silver badges37 bronze badges

Collectives™ on Stack Overflow

Gulp.js: task based on forEach loop

2 Answers 2

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related