Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

[docs] Clarify usage of Stream writable.write() #15517

Closed
wants to merge 1 commit into from
Closed

[docs] Clarify usage of Stream writable.write() #15517

wants to merge 1 commit into from

Conversation

AlexKVal
Copy link

Shame on me, but this patch is all by virtue of @misterdjules.
Thank you @misterdjules for helping me with text and code for the PR.

This PR made instead of #14002.
(for not polluting the code base by needless code from patch-1)

Shame on me, but this patch is all by virtue of @misterdjules.
Thank you @misterdjules for helping me with text and code for the PR.
@misterdjules
Copy link

LGTM, thank you @AlexKVal!

misterdjules pushed a commit to misterdjules/node that referenced this pull request Apr 15, 2015
Add separate sample code for the write-after-end case to avoid
confusion.

PR: nodejs#15517
PR-URL: nodejs#15517
Reviewed-By: Julien Gilli <jgilli@fastmail.fm>
@misterdjules
Copy link

Landed in the v0.10 branch in 9800e0b with a slightly reworded commit message. Thanks again @AlexKVal! This will be merged in v0.12 and master eventually.

jBarz pushed a commit to ibmruntimes/node that referenced this pull request Nov 4, 2016
Add separate sample code for the write-after-end case to avoid
confusion.

PR: nodejs#15517
PR-URL: nodejs#15517
Reviewed-By: Julien Gilli <jgilli@fastmail.fm>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants