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

Missing word in Stream docs #11913

Closed
jyotman opened this issue Mar 18, 2017 · 0 comments
Closed

Missing word in Stream docs #11913

jyotman opened this issue Mar 18, 2017 · 0 comments
Labels
doc Issues and PRs related to the documentations. stream Issues and PRs related to the stream subsystem.

Comments

@jyotman
Copy link
Contributor

jyotman commented Mar 18, 2017

There is a small error in the documentation for Streams. Under the Readable Streams section It's mentioned -

Note: If a Readable is switched into flowing mode and there are no consumers available handle the data, that data will be lost.

I think the word "to" is missing in that sentence between the "available" and "handle" words. I think it should be -

Note: If a Readable is switched into flowing mode and there are no consumers available to handle the data, that data will be lost.

@Fishrock123 Fishrock123 added the doc Issues and PRs related to the documentations. label Mar 18, 2017
@mscdex mscdex added the stream Issues and PRs related to the stream subsystem. label Mar 18, 2017
jungx098 pushed a commit to jungx098/node that referenced this issue Mar 21, 2017
PR-URL: nodejs#11914
Fixes: nodejs#11913
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit that referenced this issue Mar 28, 2017
PR-URL: #11914
Fixes: #11913
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit that referenced this issue Apr 18, 2017
PR-URL: #11914
Fixes: #11913
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
MylesBorins pushed a commit that referenced this issue Apr 19, 2017
PR-URL: #11914
Fixes: #11913
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
andrew749 pushed a commit to michielbaird/node that referenced this issue Jul 19, 2017
PR-URL: nodejs/node#11914
Fixes: nodejs/node#11913
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. stream Issues and PRs related to the stream subsystem.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants