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

Make pipe behave more like Readable#pipe. Fixes #449. #450

Merged
merged 2 commits into from
Feb 13, 2016

Conversation

vqvu
Copy link
Collaborator

@vqvu vqvu commented Feb 9, 2016

Pipe didn't support all of the behaviors of Readable#pipe, which results in annoying bugs. This brings it more in line.

  1. pipe now emits the pipe event on the destination when piping.
  2. pipe has an optional pipe options argument that allows users to
    choose to not end the destination when the source ends.

The second argument to pipe is optional, and generally we don't put optional arguments in stream transforms. However, since pipe is meant for interop, there's a bigger win in making it mimic the node stream interface.

- `pipe` now emits the `pipe` event on the destination when piping.
- `pipe` has an optional pipe options argument that allows users to
  choose to not end the destination when the source ends.
@vqvu vqvu force-pushed the pipe-event-and-pipe-options branch from 5854a1b to 25f8051 Compare February 9, 2016 02:08
@ronag
Copy link

ronag commented Feb 9, 2016

Awesome! This will enable me to remove a lot of workarounds!

@vqvu
Copy link
Collaborator Author

vqvu commented Feb 13, 2016

Seems to be no objections...I will merge.

vqvu added a commit that referenced this pull request Feb 13, 2016
Make pipe behave more like Readable#pipe. Fixes #449.
@vqvu vqvu merged commit ddae06e into caolan:master Feb 13, 2016
@vqvu vqvu deleted the pipe-event-and-pipe-options branch February 13, 2016 08:48
@vqvu vqvu added this to the 2.7.0 milestone Feb 29, 2016
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