Must call .resume() at the end? about gulp-istanbul HOT 3 CLOSED

That's a good question... The stream need to be flushed but you don't use the covered files objects in this case.

I'd run the question around the community, maybe there should be a gulp-flush plugin that just consume files but don't pass them through.

from gulp-istanbul.

Ok, so I guess the core issue it is a design error on this plugin part.

We pass through the instrumented files to allow coverage report from other environment than node. This mean that we build up the back pressure and it is never released as most user won't add any task after the istanbul stream.

Maybe we should make a config to control pass through:

gulp.src()
  .pipe(istanbul({ passthrough: true })

Or separate the coverage from actually hooking require, meaning the last step is an output step and won't pass through the files.

gulp.src()
  .pipe(istanbul.cover())
  .pipe(istanbul.hookRequire())

from gulp-istanbul.

Thanks, this fixed the issue for me :)

from gulp-istanbul.