★ wanayoo — archive 1999 https://github.com/nodejs/node/pull/26954Nouvelle recherche | Portail wanayoo
Skip to content
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.

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

esm: use primordials #26954

Closed
wants to merge 1 commit into from
Closed

esm: use primordials #26954

wants to merge 1 commit into from

Conversation

MylesBorins
Copy link
Member

@MylesBorins MylesBorins commented Mar 28, 2019

Converted uses of Object and Map to use frozen built
in primordials.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes

Converted uses of Object and Map to use frozen built
in primordials.
@MylesBorins MylesBorins requested a review from joyeecheung Mar 28, 2019
@MylesBorins
Copy link
Member Author

@MylesBorins MylesBorins commented Mar 28, 2019

Can we fastrack please?

@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented Mar 28, 2019

targos
targos approved these changes Mar 28, 2019
@targos
Copy link
Member

@targos targos commented Mar 30, 2019

Landed in 7a54709

@targos targos closed this Mar 30, 2019
targos added a commit that referenced this issue Mar 30, 2019
Converted uses of Object and Map to use frozen built
in primordials.

PR-URL: #26954
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Guy Bedford <guybedford@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
BethGriggs pushed a commit that referenced this issue Apr 5, 2019
Converted uses of Object and Map to use frozen built
in primordials.

PR-URL: #26954
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Guy Bedford <guybedford@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

7 participants