Combining streams recipe doesn't work

The line combined.on('error', function(err) {...}); just throws an error:

TypeError: Object function (){
    return pipeline.apply(pipeline, args);
  } has no method 'on'

Also, it would be good to see a bit more documentation about errors generally, especially in the context of gulp.watch.

👍 specially on require('gulp-util').combine versus require('event-stream').concat which seem similar.

Will take a PR to correct the docs. I'd like to change that page to use the lazypipe module instead of gulp-util. I'm traveling ATM so won't have time to do this

@contra just tell me what is the correct usage and I'll make the PR

Fix combine example

@tomasdev

@tomasdev Do you think you could modify the example to use lazypipe instead? The problem with gutil.combine is that the streams are only created once so once they end the factory will quit working

@contra you mean like https://gist.github.com/OverZealous/8503130 or multipipe? or es.concat/merge?

https://npmjs.org/package/lazypipe

Change gutil.combine to lazypipe. Fixes gulpjs#251

docs on lazypipe, docs on combine streams for errors. closes #254 #252 …

Navigation Menu

Repository navigation

Combining streams recipe doesn't work #251

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Combining streams recipe doesn't work #251

Description

Activity

callumlocke commented on Feb 7, 2014

tomasdev commented on Feb 7, 2014

yocontra commented on Feb 8, 2014

tomasdev commented on Feb 8, 2014

yocontra commented on Feb 8, 2014

tomasdev commented on Feb 8, 2014

yocontra commented on Feb 8, 2014

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions