Skip to content

Conversation

@cjihrig
Copy link
Contributor

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

doc

Description of change

cluster.setupMaster() can be called more than once. Core even has tests for this functionality. This commit removes an incorrect statement to the contrary from the documentation.

Closes#7156

@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Jun 6, 2016
@jasnell
Copy link
Member

LGTM

@mscdexmscdex added the cluster Issues and PRs related to the cluster subsystem. label Jun 6, 2016
@claudiorodriguez
Copy link
Contributor

LGTM

cluster.setupMaster() can be called more than once. Core even has tests for this functionality. This commit removes an incorrect statement to the contrary from the documentation. Fixes: nodejs#7156 PR-URL: nodejs#7179 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Claudio Rodriguez <[email protected]>
@cjihrigcjihrig merged commit 779091f into nodejs:masterJun 8, 2016
@cjihrigcjihrig deleted the docs branch June 8, 2016 15:52
evanlucas pushed a commit that referenced this pull request Jun 16, 2016
cluster.setupMaster() can be called more than once. Core even has tests for this functionality. This commit removes an incorrect statement to the contrary from the documentation. Fixes: #7156 PR-URL: #7179 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Claudio Rodriguez <[email protected]>
@evanlucasevanlucas mentioned this pull request Jun 16, 2016
MylesBorins pushed a commit that referenced this pull request Jul 11, 2016
cluster.setupMaster() can be called more than once. Core even has tests for this functionality. This commit removes an incorrect statement to the contrary from the documentation. Fixes: #7156 PR-URL: #7179 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Claudio Rodriguez <[email protected]>
MylesBorins pushed a commit that referenced this pull request Jul 12, 2016
cluster.setupMaster() can be called more than once. Core even has tests for this functionality. This commit removes an incorrect statement to the contrary from the documentation. Fixes: #7156 PR-URL: #7179 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Claudio Rodriguez <[email protected]>
MylesBorins pushed a commit that referenced this pull request Jul 12, 2016
cluster.setupMaster() can be called more than once. Core even has tests for this functionality. This commit removes an incorrect statement to the contrary from the documentation. Fixes: #7156 PR-URL: #7179 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Claudio Rodriguez <[email protected]>
@MylesBorinsMylesBorins mentioned this pull request Jul 12, 2016
MylesBorins pushed a commit that referenced this pull request Jul 14, 2016
cluster.setupMaster() can be called more than once. Core even has tests for this functionality. This commit removes an incorrect statement to the contrary from the documentation. Fixes: #7156 PR-URL: #7179 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Claudio Rodriguez <[email protected]>
MylesBorins pushed a commit that referenced this pull request Jul 14, 2016
cluster.setupMaster() can be called more than once. Core even has tests for this functionality. This commit removes an incorrect statement to the contrary from the documentation. Fixes: #7156 PR-URL: #7179 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Claudio Rodriguez <[email protected]>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clusterIssues and PRs related to the cluster subsystem.docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@cjihrig@jasnell@claudiorodriguez@mscdex@MylesBorins@nodejs-github-bot