Skip to content

src: include AsyncWrap provider strings in snapshot - #32572

Closed
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:async-wrap-providers-snapshot
Closed

addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:async-wrap-providers-snapshot

Conversation

@addaleax

Copy link
Copy Markdown
Member

… and move them to IsolateData, because they should exist once
per Isolate.

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

… and move them to `IsolateData`, because they should exist once
per Isolate.
@nodejs-github-bot nodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Mar 30, 2020
@nodejs-github-bot

nodejs-github-bot commented Mar 30, 2020 •

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/30270/ (:heavy_check_mark:)

@addaleax addaleax added the author ready PRs with CI started, the required approvals, and no outstanding review comments. label Mar 31, 2020
addaleax added a commit that referenced this pull request Apr 2, 2020
… and move them to `IsolateData`, because they should exist once
per Isolate.

PR-URL: #32572
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
@addaleax

addaleax commented Apr 2, 2020

Copy link
Copy Markdown
Member Author

Landed in 2a88515

@addaleax addaleax closed this Apr 2, 2020
@addaleax
addaleax deleted the async-wrap-providers-snapshot branch April 2, 2020 15:46
BethGriggs pushed a commit that referenced this pull request Apr 7, 2020
… and move them to `IsolateData`, because they should exist once
per Isolate.

PR-URL: #32572
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
@targos targos removed the author ready PRs with CI started, the required approvals, and no outstanding review comments. label Apr 25, 2020
targos pushed a commit to targos/node that referenced this pull request Apr 25, 2020
… and move them to `IsolateData`, because they should exist once
per Isolate.

PR-URL: nodejs#32572
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
targos pushed a commit that referenced this pull request Apr 28, 2020
… and move them to `IsolateData`, because they should exist once
per Isolate.

PR-URL: #32572
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants