Skip to content

doc: improve AsyncLocalStorage introduction#36946

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Closed
brillout wants to merge 6 commits into from
Closed

doc: improve AsyncLocalStorage introduction #36946

brillout wants to merge 6 commits into from

Conversation

@brillout
Copy link
Contributor

@brilloutbrillout commented Jan 15, 2021

I've tried to comply to the contributing guidlines although I skimed over some of it; hopefully I didn't miss anything.

@nodejs-github-botnodejs-github-bot added async_hooks Issues and PRs related to the async hooks subsystem. doc Issues and PRs related to the documentations. labels Jan 15, 2021
vdeturckheim
vdeturckheim suggested changes Jan 15, 2021
View reviewed changes
Copy link
Member

@vdeturckheimvdeturckheim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Globally LGTM, @puzpuzpuz@Flarna@Qard maybe you want to take a look :)

puzpuzpuz
puzpuzpuz reviewed Jan 15, 2021
View reviewed changes
puzpuzpuz
puzpuzpuz reviewed Jan 15, 2021
View reviewed changes
puzpuzpuz
puzpuzpuz reviewed Jan 15, 2021
View reviewed changes
Qard
Qard requested changes Jan 15, 2021
View reviewed changes
@brillout
Copy link
ContributorAuthor

brillout commented Jan 18, 2021

@vdeturckheim
@Qard
@puzpuzpuz

I rewrote and shorten the paragraph; I believe it resolves all issues.

vdeturckheim
vdeturckheim approved these changes Jan 18, 2021
View reviewed changes
Copy link
Member

@vdeturckheimvdeturckheim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

puzpuzpuz
puzpuzpuz reviewed Jan 19, 2021
View reviewed changes
@puzpuzpuzpuzpuzpuz self-requested a review January 19, 2021 13:14
Qard
Qard reviewed Jan 19, 2021
View reviewed changes
@jasnell
Copy link
Member

jasnell commented Jan 25, 2021

Landed in 63d978c

@jasnelljasnell closed this Jan 25, 2021
jasnell pushed a commit that referenced this pull request Jan 25, 2021
@brillout@jasnell
doc: improve AsyncLocalStorage introduction
63d978c
PR-URL: #36946 Reviewed-By: Vladimir de Turckheim <[email protected]> Reviewed-By: Stephen Belanger <[email protected]> Reviewed-By: Andrey Pechkurov <[email protected]> Reviewed-By: James M Snell <[email protected]>
targos pushed a commit that referenced this pull request Feb 2, 2021
@brillout@targos
doc: improve AsyncLocalStorage introduction
c79688f
PR-URL: #36946 Reviewed-By: Vladimir de Turckheim <[email protected]> Reviewed-By: Stephen Belanger <[email protected]> Reviewed-By: Andrey Pechkurov <[email protected]> Reviewed-By: James M Snell <[email protected]>
@targostargos mentioned this pull request Feb 2, 2021
Merged
targos pushed a commit that referenced this pull request May 1, 2021
@brillout@targos
doc: improve AsyncLocalStorage introduction
49f1374
PR-URL: #36946 Reviewed-By: Vladimir de Turckheim <[email protected]> Reviewed-By: Stephen Belanger <[email protected]> Reviewed-By: Andrey Pechkurov <[email protected]> Reviewed-By: James M Snell <[email protected]>
@danielleadamsdanielleadams mentioned this pull request May 3, 2021
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@QardQardQard approved these changes

@jasnelljasnelljasnell approved these changes

+2 more reviewers

@puzpuzpuzpuzpuzpuzpuzpuzpuz approved these changes

@vdeturckheimvdeturckheimvdeturckheim approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

async_hooksIssues and PRs related to the async hooks subsystem.docIssues and PRs related to the documentations.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@brillout@jasnell@Qard@vdeturckheim@puzpuzpuz@nodejs-github-bot