-
Notifications
You must be signed in to change notification settings - Fork 29.7k
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: add added:
information for child_process
#6927
Conversation
@@ -113,6 +113,9 @@ exec('my.bat', (err, stdout, stderr) => { | |||
``` | |||
|
|||
### child_process.exec(command[, options][, callback]) | |||
<!-- YAML | |||
added: v0.1.33 | |||
--> |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
5f4764d
to
9b0a754
Compare
@@ -231,6 +237,9 @@ stderr output. If `encoding` is `'buffer'`, `Buffer` objects will be passed to | |||
the callback instead. | |||
|
|||
### child_process.fork(modulePath[, args][, options]) | |||
<!-- YAML | |||
added: v0.4.8 |
There was a problem hiding this comment.
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.
9b0a754
to
cd91ce8
Compare
/cc @nodejs/documentation |
LGTM as long as @bnoordhuis is happy |
Yes, LGTM. The two errors I pointed out were the only ones. |
Landed in 27d2267 |
Ref: nodejs#6578 PR-URL: nodejs#6927 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Ref: nodejs#6578 PR-URL: nodejs#6927 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Checklist
Affected core subsystem(s)
doc
Description of change
Ref: #6578