Skip to content

Conversation

rluvaton
Copy link
Member

No description provided.

@nodejs-github-bot nodejs-github-bot added async_hooks Issues and PRs related to the async hooks subsystem. doc Issues and PRs related to the documentations. labels Jul 24, 2021
@rluvaton rluvaton changed the title docs(async-hooks): correct cjs code to match mjs code doc(async-hooks): correct cjs code to match mjs code Jul 24, 2021
@rluvaton
Copy link
Member Author

rluvaton commented Jul 28, 2021

How many reviewers need to approve before a PR can be merged? 😀

@lpinca
Copy link
Member

lpinca commented Aug 16, 2021

Landed in e087d78.

lpinca pushed a commit that referenced this pull request Aug 16, 2021
PR-URL: #39509
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Harshitha K P <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Vladimir de Turckheim <[email protected]>
@lpinca lpinca closed this Aug 16, 2021
@rluvaton rluvaton deleted the patch-1 branch August 16, 2021 14:06
@rluvaton
Copy link
Member Author

Thank you @lpinca
Is there a reason why this PR flagged as closed rather then merged?

@lpinca
Copy link
Member

lpinca commented Aug 16, 2021

That's how PRs are merged in Node.js core. See https://github.com/nodejs/node/blob/966060df0e4aa7d79bfdd9fe2163b12298cf59b7/doc/guides/collaborator-guide.md#landing-pull-requests.

danielleadams pushed a commit that referenced this pull request Aug 16, 2021
PR-URL: #39509
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Harshitha K P <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Vladimir de Turckheim <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

async_hooks Issues and PRs related to the async hooks subsystem. doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants