-
Notifications
You must be signed in to change notification settings - Fork 29.1k
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, stream: readable._destroy() description is missing in the stream.md #15291
Comments
vsemozhetbyt
added
doc
Issues and PRs related to the documentations.
stream
Issues and PRs related to the stream subsystem.
labels
Sep 9, 2017
Definitely, I'll get this added asap. This is a also a first good contribution, if someone wants to take it. The content for |
mcollina
added
the
good first issue
Issues that are suitable for first-time contributors.
label
Sep 10, 2017
2 tasks
Refs: #15316 |
targos
removed
the
good first issue
Issues that are suitable for first-time contributors.
label
Sep 10, 2017
targos
added a commit
to targos/node
that referenced
this issue
Sep 10, 2017
addaleax
pushed a commit
to addaleax/ayo
that referenced
this issue
Sep 17, 2017
PR-URL: nodejs/node#15316 Fixes: nodejs/node#15291 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
jasnell
pushed a commit
that referenced
this issue
Sep 20, 2017
PR-URL: #15316 Fixes: #15291 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Qard
pushed a commit
to Qard/ayo
that referenced
this issue
Sep 21, 2017
PR-URL: nodejs/node#15316 Fixes: nodejs/node#15291 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There is a link to
readable._destroy()
in thestream.md
after 330c8d7, but there is noreadable._destroy()
description in the Implementing a Readable Stream part.cc @mcollina
The text was updated successfully, but these errors were encountered: