Should filtering be implemented as an option here? about glob-stream HOT 5 CLOSED

@phated Yeah, I agree. Do you want to expose that option in vinyl-fs though? How about async filter functions + stream ordering?

from glob-stream.

I think vinyl-fs still exposes since option and we generate the filter method and pass it to glob-stream like globStream.create(globs, { filter: sinceFilter }). I guess we could also expose filter directly like { filter: options.filter || sinceFilter }

Not sure how we'd do stream ordering though.

from glob-stream.

It might make sense to have a filter option here regardless, but I think that when I pass a function as since option to vinylfs.src(), I would expect it to be invoked with an actual vinyl argument, as the other options are (or will be).

So I don't think vinyl-fs/vinyl-prepare can simply hand this option off to glob-stream, because the corresponding vinyl object will not yet have been created at that time...?

from glob-stream.

@erikkemperman good catch, we'll probably not push filter down into this project then.

from glob-stream.

Going to close this since it doesn't feel appropriate to implement here. Might revisit in the future if alternative forms of filtering seem to make sense here.

from glob-stream.