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.

Sign up for GitHub

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

Jump to bottom

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.
@jyotman jyotman mentioned this issue Mar 18, 2017
Closed
2 tasks
@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
@jyotman @jungx098
doc: add missing word in stream.md
e8d9da8 
PR-URL: nodejs#11914
Fixes: nodejs#11913
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this issue Mar 28, 2017
@jyotman @MylesBorins
doc: add missing word in stream.md
68b23be 
PR-URL: #11914
Fixes: #11913
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this issue Apr 18, 2017
@jyotman @MylesBorins
doc: add missing word in stream.md
88aa556 
PR-URL: #11914
Fixes: #11913
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this issue Apr 19, 2017
@jyotman @MylesBorins
doc: add missing word in stream.md
4a97bc7 
PR-URL: #11914
Fixes: #11913
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
andrew749 pushed a commit to michielbaird/node that referenced this issue Jul 19, 2017
@jyotman @andrew749
doc: add missing word in stream.md
f02c56e 
PR-URL: nodejs/node#11914
Fixes: nodejs/node#11913
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
Reviewed-By: Timothy Gu <timothygu99@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
doc Issues and PRs related to the documentations. stream Issues and PRs related to the stream subsystem.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

doc: add missing word in stream.md
3 participants
@mscdex @Fishrock123 @jyotman