-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[DOC] Add inline docs to stream util methods and rename chainStream to chain. #10105
Conversation
Looks good.
|
Thanks @mmun I'll mark them private for now and we can decide whether to expose them as public API over time. Also will rename |
036c39a
to
f15f9f7
Compare
OK, all set from my perspective (pending a green build!). |
f15f9f7
to
7ef2dc1
Compare
7ef2dc1
to
f0db31e
Compare
|
||
@private | ||
@function chain | ||
@param {Object|Stream} array An array containing zero or more stream objects |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These aren't right.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
D'oh!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops, good catch. Will fix.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix in #10107
[DOC] Add inline docs to stream util methods and rename chainStream to chain.
Thanks Luke! |
@mixonic spin-off of your pointers about the non-stream-safe util methods from #10093