Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add pipe() #95

Closed
wants to merge 2 commits into from
Closed

add pipe() #95

wants to merge 2 commits into from

Conversation

icambron
Copy link

Proposed fix for #94

@Rich-Harris
Copy link
Contributor

I like this! How about adding an options argument so that you can do this sort of thing?

module.exports = gobble([
  gobble( 'src/a' ).pipe( x, { id: 'a' }),
  gobble( 'src/b' ).pipe( x, { id: 'b' })
]);

That way you could more neatly encapsulate things like moving the resulting node to a specific directory.

@icambron
Copy link
Author

Done!

@icambron
Copy link
Author

icambron commented Dec 7, 2015

(Also, I do see that the build is failing on Appveyor, but if I understand the output correctly, it's an environment/Node-building issue, and not a problem with this PR)

@icambron icambron closed this Apr 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants