search

tschaub / gulp-newer

Pass through newer source files only
https://npmjs.org/package/gulp-newer
226 stars 24 forks source link

Add options.map #12

Closed karlvr closed 9 years ago

tschaub commented 9 years ago

Nice. One thing to consider is async name resolution (e.g. https://github.com/tschaub/gulp-newer/issues/1#issuecomment-45984636). I'd like to think about how common those use cases are.

karlvr commented 9 years ago

I'm not sure I understand the idea of async name resolution. The map function definitely yields a lot of flexibility. I'm not sure what other parameters may be useful to the function - this PR is a pretty simple implementation. Could you explain async name resolution?

tschaub commented 9 years ago

Thanks for the work on this @karlvr. I don't have any use cases for async name resolution right now. If someone comes up with a compelling one in the future, we could rely on the length of the map function to check for a callback.

karlvr commented 9 years ago

Thanks @tschaub, my pleasure.