FR : Filters about gulp HOT 12 CLOSED

This sounds like a good user-land module!

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

module.exports = function(opt){
  return es.map(function(file, cb){
    if (file.stat && file.stat.isDirectory() && opt.isFile) return cb(); // filter it out
    cb(null, file); // pass it through
  });
};

A good filter module with support for globs would actually be a good replacement for having filtering and glob negation in core #6 #7

from gulp.

@contra Neat!

But, would it not be neat for gulp if it integrate glob negation ?

from gulp.

@hemanth - Yeah... glob negation will be in core. All other filters will be user-land. One main gulp-filter module would be really badass. I'll put up a $5 bounty to whoever wants to make it.

from gulp.

I've got a module that does this now. I'll publish shortly.

Rob
On Dec 2, 2013 8:53 PM, "Eric Schoffstall" [email protected] wrote:

@hemanth https://github.com/hemanth - Glob negation will be in core.
All other filters will be user-land. One main gulp-filter module would be
really badass. I'll put up a $5 bounty to whoever wants to make it

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/35#issuecomment-29681701
.

from gulp.

@robrich Making code and... not open sourcing it immediately? 😖

from gulp.

@robrich Seems like @robrich was waiting to win the $5 bounty 🌵 👍

from gulp.

Yeah, fail on me. I wasn't sure if it was valuable, so I didn't bother. Seems it now is. :D

from gulp.

Check out https://npmjs.org/package/gulp-ignore

from gulp.

Nice it supports globs! Closing this as it seems that your module solves the issue. Glob negation will be in core soon folks

from gulp.

@contra @robrich Sweet! :)

from gulp.

Yes, glob negation should be in core. I keep hitting Error: EMFILE, too many open files. I can't use @robrich's gulp-ignore because it just won't get that far.

from gulp.

@hparra - Working on it. Ironing out how it should behave is taking longer than expected (lots of edge cases when dealing with globs)

from gulp.