FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

doc: cleanup isDead() example by cjihrig · Pull Request #28421 · nodejs/node · GitHub

/ node Public

doc: cleanup isDead() example - #28421

Merged
cjihrig merged 1 commit into
nodejs:masterfrom
cjihrig:cluster
Jun 27, 2019
Merged

doc: cleanup isDead() example#28421
cjihrig merged 1 commit into
nodejs:masterfrom
cjihrig:cluster

Conversation

cjihrig commented Jun 25, 2019

Copy link
Copy Markdown
Contributor

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 added cluster Issues and PRs related to the cluster subsystem. doc Issues and PRs related to the documentations. labels Jun 25, 2019

Trott commented Jun 25, 2019
edited
Loading

Copy link
Copy Markdown
Member

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

trivikr commented Jun 25, 2019

Copy link
Copy Markdown
Member

Feel free to 👍 this comment to approve fast-tracking

trivikr added the fast-track PRs that do not need to wait for 48 hours to land. label Jun 25, 2019

cjihrig commented Jun 26, 2019

Copy link
Copy Markdown
Contributor Author

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 merged commit 63a5cd8 into nodejs:master Jun 27, 2019
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 mentioned this pull request Jul 2, 2019
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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


Back | FazBrowse Home | New Git URL