Skip to content

doc: add added: information for child_process - #6927

Closed
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:doc-added-child_process
Closed

doc: add added: information for child_process#6927
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:doc-added-child_process

Conversation

@addaleax

Copy link
Copy Markdown
Member
Checklist
  • documentation is changed or added
  • the commit message follows commit guidelines
Affected core subsystem(s)

doc

Description of change

Ref: #6578

@addaleaxaddaleax added child_process Issues and PRs related to the child_process subsystem. doc Issues and PRs related to the documentations. labels May 23, 2016
Comment threaddoc/api/child_process.md Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wasn't this util.exec() at that time?

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

04c06b9 uses require("child_process").exec in the docs, that’s pretty much what I went by here. But yeah, sorry, this is one version off.

@addaleax
addaleaxforce-pushed the doc-added-child_process branch from 5f4764d to 9b0a754CompareMay 23, 2016 15:31
Comment threaddoc/api/child_process.md Outdated

@bnoordhuisbnoordhuisMay 23, 2016

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Close. It was added in v0.5.0.

@addaleax
addaleaxforce-pushed the doc-added-child_process branch from 9b0a754 to cd91ce8CompareMay 23, 2016 19:30
@addaleax

Copy link
Copy Markdown
MemberAuthor

/cc @nodejs/documentation

@jasnell

Copy link
Copy Markdown
Member

LGTM as long as @bnoordhuis is happy

@bnoordhuis

Copy link
Copy Markdown
Member

Yes, LGTM. The two errors I pointed out were the only ones.

@addaleax

Copy link
Copy Markdown
MemberAuthor

Landed in 27d2267

@addaleax
addaleax deleted the doc-added-child_process branch May 27, 2016 02:31
addaleax added a commit that referenced this pull request May 27, 2016
Ref: #6578
PR-URL: #6927
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Fishrock123 pushed a commit to Fishrock123/node that referenced this pull request May 30, 2016
Ref: nodejs#6578
PR-URL: nodejs#6927
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
rvagg pushed a commit that referenced this pull request Jun 2, 2016
Ref: #6578
PR-URL: #6927
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
addaleax added a commit to addaleax/node that referenced this pull request Jul 12, 2016
Ref: nodejs#6578
PR-URL: nodejs#6927
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
MylesBorins pushed a commit that referenced this pull request Jul 12, 2016
Ref: #6578
PR-URL: #6927
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
MylesBorins pushed a commit that referenced this pull request Jul 12, 2016
Ref: #6578
PR-URL: #6927
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
@MylesBorinsMylesBorins mentioned this pull request Jul 12, 2016
MylesBorins pushed a commit that referenced this pull request Jul 14, 2016
Ref: #6578
PR-URL: #6927
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
MylesBorins pushed a commit that referenced this pull request Jul 14, 2016
Ref: #6578
PR-URL: #6927
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

child_processIssues and PRs related to the child_process subsystem.docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@addaleax@jasnell@bnoordhuis@Fishrock123@MylesBorins