-
Notifications
You must be signed in to change notification settings - Fork 30
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
combinedStream.pipe() not working #1
Comments
can you post runnable code? and is zed a readable stream? |
@dominictarr Hm, this contrived test works:
I must now review my use-case. |
hmm, that would work if you used through instead of passthrough... if you really want to use stream2, I think this module: https://npmjs.org/package/bun is the same idea as stream-combiner but for streams2. |
@dominictarr thanks for the link. I did not know about bun. What do you advocate as the best approach to streams for someone new to them? Asking them not to become familiar and comfortable with the natives seems like a stretch. |
@dominictarr bun works so I assume that the issue is simply a node version incompatibility and not a bug in stream-combiner per-se. Feel free to close. You may want to update your readme with support information. |
The
pipe
tomoo
does not work (node 0.10.x).The text was updated successfully, but these errors were encountered: