Skip to content

doc,worker_threads: revise worker_threads.md - #25402

Closed
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:revise-worker-threads-doc
Closed

doc,worker_threads: revise worker_threads.md#25402
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:revise-worker-threads-doc

Conversation

@Trott

@TrottTrott commented Jan 8, 2019

Copy link
Copy Markdown
Member

Rewrite worker_threads introductory material focusing on simpler and
shorter sentences.

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

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Jan 8, 2019
Comment threaddoc/api/worker_threads.md Outdated
Comment threaddoc/api/worker_threads.md Outdated
Rewrite worker_threads introductory material focusing on simpler and
shorter sentences.
@Trott
Trottforce-pushed the revise-worker-threads-doc branch from a13e303 to 3bd05fbCompareJanuary 9, 2019 03:01
@Trott

Trott commented Jan 9, 2019

Copy link
Copy Markdown
MemberAuthor

Rebased, updated based on nits, force-pushed.

@Trott

Trott commented Jan 9, 2019

Copy link
Copy Markdown
MemberAuthor

@TrottTrott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jan 9, 2019
@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in 1415d57

@TrottTrott closed this Jan 11, 2019
Trott added a commit to Trott/io.js that referenced this pull request Jan 11, 2019
Rewrite worker_threads introductory material focusing on simpler and
shorter sentences.
PR-URL: nodejs#25402
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Eugene Ostroukhov <eostroukhov@google.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
addaleax pushed a commit that referenced this pull request Jan 14, 2019
Rewrite worker_threads introductory material focusing on simpler and
shorter sentences.
PR-URL: #25402
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Eugene Ostroukhov <eostroukhov@google.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@BridgeARBridgeAR mentioned this pull request Jan 16, 2019
BridgeAR pushed a commit to BridgeAR/node that referenced this pull request Jan 16, 2019
Rewrite worker_threads introductory material focusing on simpler and
shorter sentences.
PR-URL: nodejs#25402
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Eugene Ostroukhov <eostroukhov@google.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request Jan 24, 2019
@Trott
Trott deleted the revise-worker-threads-doc branch January 13, 2022 22:50
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.docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@Trott@nodejs-github-bot@eugeneo@jasnell@addaleax@lpinca@cjihrig