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

Add anchors for _transform _flush _writev #4448

Closed
wants to merge 1 commit into from

Conversation

iamchenxin
Copy link
Contributor

add the anchors for _transform _flush _writev in (API for Stream Implementors) table.

@vkurchatkin vkurchatkin added the doc Issues and PRs related to the documentations. label Dec 28, 2015
@trevnorris
Copy link
Contributor

LGTM

@mscdex mscdex added the stream Issues and PRs related to the stream subsystem. label Dec 28, 2015
@jasnell
Copy link
Member

jasnell commented Dec 30, 2015

LGTM

jasnell pushed a commit that referenced this pull request Dec 30, 2015
PR-URL: #4448
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@jasnell
Copy link
Member

jasnell commented Dec 30, 2015

Landed in 1de6e4f

@rvagg
Copy link
Member

rvagg commented Jan 4, 2016

Great addition @iamchenxin, thanks for this. I think this is your first commit to core, welcome on board, let us know if you need help making additional contributions!

@iamchenxin
Copy link
Contributor Author

@rvagg Thanks,i will and i should . I am a newbie to node, will constant learning and feedback.

Fishrock123 pushed a commit to Fishrock123/node that referenced this pull request Jan 6, 2016
PR-URL: nodejs#4448
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jan 13, 2016
PR-URL: #4448
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
MylesBorins pushed a commit that referenced this pull request Jan 19, 2016
PR-URL: #4448
Reviewed-By: Trevor Norris <trev.norris@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@MylesBorins MylesBorins mentioned this pull request Jan 19, 2016
scovetta pushed a commit to scovetta/node that referenced this pull request Apr 2, 2016
PR-URL: nodejs#4448
Reviewed-By: Trevor Norris <trev.norris@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
Labels
doc Issues and PRs related to the documentations. stream Issues and PRs related to the stream subsystem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants