Skip to content

doc: cleanup isDead() example #28421

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jun 27, 2019
Merged

doc: cleanup isDead() example #28421

merged 1 commit into from
Jun 27, 2019

Conversation

cjihrig
Copy link
Contributor

@cjihrig cjihrig commented Jun 25, 2019

This commit removes extra whitespace and some awkward text containing typos from the cluster worker.isDead() code sample.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added cluster Issues and PRs related to the cluster subsystem. doc Issues and PRs related to the documentations. labels Jun 25, 2019
@Trott
Copy link
Member

Trott commented Jun 25, 2019

Super micro-nit: s/cleanup/clean up/ in commit message?

@trivikr
Copy link
Member

trivikr commented Jun 25, 2019

Feel free to 👍 this comment to approve fast-tracking

@trivikr trivikr added the fast-track PRs that do not need to wait for 48 hours to land. label Jun 25, 2019
@cjihrig
Copy link
Contributor Author

cjihrig commented Jun 26, 2019

This commit removes extra whitespace and some awkward text
containing typos from the cluster worker.isDead() code
sample.

PR-URL: nodejs#28421
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@cjihrig cjihrig merged commit 63a5cd8 into nodejs:master Jun 27, 2019
@cjihrig cjihrig deleted the cluster branch June 27, 2019 13:52
targos pushed a commit that referenced this pull request Jul 2, 2019
This commit removes extra whitespace and some awkward text
containing typos from the cluster worker.isDead() code
sample.

PR-URL: #28421
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this pull request Jul 2, 2019
This commit removes extra whitespace and some awkward text
containing typos from the cluster worker.isDead() code
sample.

PR-URL: #28421
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@targos targos mentioned this pull request Jul 2, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
cluster Issues and PRs related to the cluster subsystem. doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants