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

net, child_process, and buffer doc patches #2903

Closed
wants to merge 3 commits into from

Conversation

sam-github
Copy link
Contributor

Fix some minor errors and omissions in docs

@sam-github sam-github added the doc Issues and PRs related to the documentations. label Sep 16, 2015
@mscdex mscdex added buffer Issues and PRs related to the buffer subsystem. child_process Issues and PRs related to the child_process subsystem. net Issues and PRs related to the net subsystem. labels Sep 16, 2015
@@ -366,7 +366,8 @@ callback or returning an EventEmitter).
* `env` {Object} Environment key-value pairs
* `stdio` {Array|String} Child's stdio configuration. (See
[below](#child_process_options_stdio))
* `detached` {Boolean} The child will be a process group leader. (See
* `detached` {Boolean} Prepare child to run independently of it's parent
Copy link
Contributor

Choose a reason for hiding this comment

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

No apostrophe in its.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks

@sam-github
Copy link
Contributor Author

Fixed those two things, otherwise LGTY?

@cjihrig
Copy link
Contributor

cjihrig commented Sep 16, 2015

Yea, LGTM with one question. None of the child_process changes depend on #2908 right?

@sam-github
Copy link
Contributor Author

They do not, the docs here describe the existing behaviour.

#2908 depends on this because .hide allows the window popup to be disabled (it comes about from me noticing this before now undocumented behaviour of detached on Windows).

sam-github added a commit that referenced this pull request Sep 16, 2015
PR-URL: #2903
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
sam-github added a commit that referenced this pull request Sep 16, 2015
PR-URL: #2903
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
sam-github added a commit that referenced this pull request Sep 16, 2015
createServer() can create socket of types other than TCP.

PR-URL: #2903
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@sam-github
Copy link
Contributor Author

Landed in 6cb9e8a, e148dcc, and a12628b.

@sam-github sam-github closed this Sep 16, 2015
sam-github added a commit that referenced this pull request Sep 16, 2015
PR-URL: #2903
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
sam-github added a commit that referenced this pull request Sep 16, 2015
PR-URL: #2903
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
sam-github added a commit that referenced this pull request Sep 16, 2015
createServer() can create socket of types other than TCP.

PR-URL: #2903
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@rvagg rvagg mentioned this pull request Sep 22, 2015
@sam-github sam-github deleted the misc-doc-fixes branch April 17, 2017 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
buffer Issues and PRs related to the buffer subsystem. child_process Issues and PRs related to the child_process subsystem. doc Issues and PRs related to the documentations. net Issues and PRs related to the net subsystem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants