feat(es/preset-env): Update preset-env data #9573
Conversation
🦋 Changeset detectedLatest commit: e402f9e The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
| ["esnext.composite-symbol", SYMBOL_DEPENDENCIES] | ||
| ), | ||
| fetch: define(null, PROMISE_DEPENDENCIES), | ||
| globalThis: define("global-this", ["es.global-this", "esnext.global-this"]), |
There was a problem hiding this comment.
The source of these new codes are https://github.com/babel/babel/blob/e498bee10f0123bb208baa228ce6417542a2c3c4/packages/babel-preset-env/src/polyfills/corejs3/built-in-definitions.js but I'm not sure if this is correct.
There was a problem hiding this comment.
It's obsolete definitions. Actual definitions here https://github.com/babel/babel-polyfills/blob/main/packages/babel-plugin-polyfill-corejs3/src/built-in-definitions.ts
CodSpeed Performance ReportMerging #9573 will degrade performances by 7.75%Comparing Summary
Benchmarks breakdown
|
![kodiakhq[bot]](https://avatars.githubusercontent.com/in/29196?s=60&v=4){kind=small}
| @@ -1,6 +1,6 @@ | |||
| require("core-js/modules/es.array.at.js"); | |||
| require("core-js/modules/es.regexp.exec.js"); | |||
| require("core-js/modules/es.string.at-alternative.js"); | |||
| require("core-js/modules/es.string.at.js"); | |||
There was a problem hiding this comment.
core-js doesn't have a module with this name.
| require("core-js/modules/es.object.to-string.js"); | ||
| document.querySelectorAll('div').forEach(function(el) { | ||
| console.log(el); | ||
| }); |
There was a problem hiding this comment.
You are right. I'll reopen it.
Description:
Related issue: