Skip to content

doc: modernize and simplify cluster example - #41626

Merged
nodejs-github-bot merged 1 commit into
nodejs:masterfrom
tniessen:doc-simplify-cluster-workers-example
Jan 23, 2022
Merged

doc: modernize and simplify cluster example#41626
nodejs-github-bot merged 1 commit into
nodejs:masterfrom
tniessen:doc-simplify-cluster-workers-example

Conversation

@tniessen

Copy link
Copy Markdown
Member

Vastly simplify the cluster.workers example code using for (... of Object.values(cluster.workers)).

@nodejs-github-botnodejs-github-bot added cluster Issues and PRs related to the cluster subsystem. doc Issues and PRs related to the documentations. labels Jan 21, 2022
@tniessen
tniessenforce-pushed the doc-simplify-cluster-workers-example branch from 2e71efe to aeae56fCompareJanuary 21, 2022 11:51
@MesteeryMesteery added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jan 21, 2022
@tniessentniessen added the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 23, 2022
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 23, 2022
@nodejs-github-bot
nodejs-github-bot merged commit 0e6db9c into nodejs:masterJan 23, 2022
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 0e6db9c

BethGriggs pushed a commit that referenced this pull request Jan 25, 2022
PR-URL: #41626
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@ruyadornoruyadorno mentioned this pull request Feb 8, 2022
danielleadams pushed a commit that referenced this pull request Feb 28, 2022
PR-URL: #41626
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
danielleadams pushed a commit that referenced this pull request Mar 2, 2022
PR-URL: #41626
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
danielleadams pushed a commit that referenced this pull request Mar 3, 2022
PR-URL: #41626
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
danielleadams pushed a commit that referenced this pull request Mar 14, 2022
PR-URL: #41626
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.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

@tniessen@nodejs-github-bot@Trott@benjamingr@lpinca@Mesteery