Skip to content

Conversation

matejkrajcovic
Copy link
Contributor

Backport of #13769.

Checklist
Affected core subsystem(s)

doc

PR-URL: nodejs#13769
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. v6.x labels Oct 27, 2017
Copy link
Member

@gibfahn gibfahn left a comment

Choose a reason for hiding this comment

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

There are a couple of bits that should probably be wrapped at 80 columns, but otherwise LGTM.

Thank you so much for doing this!

* `encoding` {string} **Default:** `'utf8'`
* `timeout` {number} **Default:** `0`
* [`maxBuffer`][] {number} Largest amount of data (in bytes) allowed on
stdout or stderr - if exceeded child process is killed. **Default:*: `200*1024`
Copy link
Member

Choose a reason for hiding this comment

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

Missing the second * after Default here.

@matejkrajcovic
Copy link
Contributor Author

@gibfahn Both issues are fixed.

@matejkrajcovic
Copy link
Contributor Author

@gibfahn Please look at this again. Is there anything I should change?

MylesBorins pushed a commit that referenced this pull request Nov 3, 2017
Backport-PR-URL: #16560
PR-URL: #13769
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
@MylesBorins
Copy link
Contributor

landed in 0f3901a

@MylesBorins MylesBorins closed this Nov 3, 2017
@matejkrajcovic matejkrajcovic deleted the backport-13769-to-v6.x branch November 3, 2017 18:04
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants