Skip to content

Conversation

@sabakugaara
Copy link
Contributor

Checklist
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

doc

In process warning event sample code, it should be events.defaultMaxListeners

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. process Issues and PRs related to the process subsystem. labels Mar 10, 2017
@aqrln
Copy link
Contributor

commit message follows commit guidelines

It actually doesn't. The first line of the commit message should not exceed 50 characters. I'd suggest to reword it to something like doc: fix a typo in api/process.md and elaborate on that in the full message.

@sabakugaarasabakugaaraforce-pushed the events-typo-for-process branch from 66da4f4 to ee10e6eCompareMarch 10, 2017 03:19
@sabakugaarasabakugaara changed the title doc: fix typo 'event' to 'events' in process module docdoc: fix a typo in api/process.mdMar 10, 2017
@sabakugaara
Copy link
ContributorAuthor

Thanks!

@sabakugaarasabakugaaraforce-pushed the events-typo-for-process branch from ee10e6e to 8b23adfCompareMarch 10, 2017 03:26
@aqrln
Copy link
Contributor

aqrln commented Mar 10, 2017

@sabakugaara glad to help :) I'd also suggest you to make your full commit message more explicit because it isn't very clear, to be honest. How about this?

Fix a mistyped module name in example REPL sessions found in the description of the 'warning' event: it should be `events` instead of `event`. 

Feel free to reword it as you like.

Copy link
Member

@TrottTrott left a comment

Choose a reason for hiding this comment

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

LGTM

Fix a mistyped module name in example REPL sessions found in the description of the 'warning' event: it should be `events` instead of `event`.
@sabakugaarasabakugaaraforce-pushed the events-typo-for-process branch from 8b23adf to 05300e0CompareMarch 10, 2017 05:31
jasnell pushed a commit that referenced this pull request Mar 15, 2017
Fix a mistyped module name in example REPL sessions found in the description of the 'warning' event: it should be `events` instead of `event`. PR-URL: #11780 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
@jasnell
Copy link
Member

Landed in 27f4c94

@jasnelljasnell closed this Mar 15, 2017
italoacasas pushed a commit to italoacasas/node that referenced this pull request Mar 20, 2017
Fix a mistyped module name in example REPL sessions found in the description of the 'warning' event: it should be `events` instead of `event`. PR-URL: nodejs#11780 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
@italoacasasitaloacasas mentioned this pull request Mar 20, 2017
jungx098 pushed a commit to jungx098/node that referenced this pull request Mar 21, 2017
Fix a mistyped module name in example REPL sessions found in the description of the 'warning' event: it should be `events` instead of `event`. PR-URL: nodejs#11780 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
MylesBorins pushed a commit that referenced this pull request Apr 18, 2017
Fix a mistyped module name in example REPL sessions found in the description of the 'warning' event: it should be `events` instead of `event`. PR-URL: #11780 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
MylesBorins pushed a commit that referenced this pull request Apr 19, 2017
Fix a mistyped module name in example REPL sessions found in the description of the 'warning' event: it should be `events` instead of `event`. PR-URL: #11780 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
@MylesBorinsMylesBorins mentioned this pull request Apr 19, 2017
andrew749 pushed a commit to michielbaird/node that referenced this pull request Jul 19, 2017
Fix a mistyped module name in example REPL sessions found in the description of the 'warning' event: it should be `events` instead of `event`. PR-URL: nodejs/node#11780 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.processIssues and PRs related to the process subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@sabakugaara@aqrln@jasnell@Trott@lpinca@cjihrig@MylesBorins@nodejs-github-bot