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

Re-generate preset-env fixtures #11016

Merged
merged 3 commits into from Jan 16, 2020

Conversation

nicolo-ribaudo
Copy link
Member

Q                       A
Tests Added + Pass? Yes
Documentation PR Link
Any Dependency Changes?
License MIT

This PR should only re-order the logged plugins. I don't know why GitHub shows more created lines than deleted, but locally the match:

➜ git diff --numstat HEAD^..HEAD

10	10	packages/babel-preset-env/test/fixtures/corejs2/usage-browserslist-config-ignore/stdout.txt
10	10	packages/babel-preset-env/test/fixtures/corejs3/usage-browserslist-config-ignore/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/browserslists-android-3/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/browserslists-defaults-not-ie/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/browserslists-defaults/stdout.txt
7	7	packages/babel-preset-env/test/fixtures/debug/browserslists-last-2-versions-not-ie/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/corejs-without-usebuiltins/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-android/stdout.txt
15	15	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-electron/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-force-all-transforms/stdout.txt
9	9	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-no-import/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-proposals-chrome-71/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-proposals/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-shippedProposals-chrome-71/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-shippedProposals/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-specific-targets/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-versions-decimals/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-versions-strings/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/entry-corejs2/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-all-chrome-71/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-all/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-android/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-babel-polyfill/stdout.txt
15	15	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-electron/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-es-chrome-71/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-es-proposals-chrome-71/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-es-proposals/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-es/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-force-all-transforms/stdout.txt
9	9	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-no-import/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-proposals-chrome-71/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-proposals/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-runtime-only-chrome-71/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-runtime-only/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-specific-entries-chrome-71/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-specific-entries/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-specific-targets/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-stable-chrome-71/stdout.txt
3	3	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-stable-samsung-8.2/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-stable/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-stage-chrome-71/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-stage/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-versions-decimals/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-versions-strings-minor-3.0/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-versions-strings-minor-3.1/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-versions-strings/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-web-chrome-71/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-web/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/entry-corejs3/stdout.txt
9	9	packages/babel-preset-env/test/fixtures/debug/entry-no-corejs-no-import/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-no-corejs-shippedProposals/stdout.txt
25	25	packages/babel-preset-env/test/fixtures/debug/entry-no-corejs-uglify/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/entry-no-corejs/stdout.txt
8	8	packages/babel-preset-env/test/fixtures/debug/plugins-only/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-1/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-2/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-chrome-71-1/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-chrome-71-2/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-none-1/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-none-2/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-proposals-1/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-proposals-2/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-proposals-chrome-71-1/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-proposals-chrome-71-2/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-shippedProposals-1/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-shippedProposals-2/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-with-import/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-1/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-2/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-chrome-71-1/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-chrome-71-2/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-none-1/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-none-2/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-proposals-1/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-proposals-2/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-proposals-chrome-71-1/stdout.txt
2	2	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-proposals-chrome-71-2/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-shippedProposals-1/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-shippedProposals-2/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-versions-strings-minor-3.0-1/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-versions-strings-minor-3.0-2/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-versions-strings-minor-3.1-1/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-versions-strings-minor-3.1-2/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-with-import/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-no-corejs-1/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-no-corejs-2/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-no-corejs-none-1/stdout.txt
24	24	packages/babel-preset-env/test/fixtures/debug/usage-no-corejs-none-2/stdout.txt
9	9	packages/babel-preset-env/test/fixtures/preset-options/safari-10_3-block-scoped/stdout.txt
15	10	packages/babel-preset-env/test/fixtures/sanity/block-scoping-for-of/output.js

@nicolo-ribaudo
Copy link
Member Author

Oh nevermind, packages/babel-preset-env/test/fixtures/sanity/block-scoping-for-of/output.js changed. It seems caused by a different plugin ordering, introduced because of https://github.com/babel/babel/pull/10811/files#diff-a22feee84e9f4b2f880aabb7595a1c87 which reordered the plugins.

@nicolo-ribaudo
Copy link
Member Author

This is not safe, I'll investigate

@nicolo-ribaudo
Copy link
Member Author

nicolo-ribaudo commented Jan 16, 2020

Ok now it looks ok:

201	201	packages/babel-compat-data/data/plugins.json
92	68	packages/babel-compat-data/scripts/data/plugin-features.js
10	10	packages/babel-preset-env/test/fixtures/corejs2/usage-browserslist-config-ignore/stdout.txt
10	10	packages/babel-preset-env/test/fixtures/corejs3/usage-browserslist-config-ignore/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/browserslists-android-3/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/browserslists-defaults-not-ie/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/browserslists-defaults/stdout.txt
8	8	packages/babel-preset-env/test/fixtures/debug/browserslists-last-2-versions-not-ie/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/corejs-without-usebuiltins/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-android/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-electron/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-force-all-transforms/stdout.txt
10	10	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-no-import/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-proposals-chrome-71/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-proposals/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-shippedProposals-chrome-71/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-shippedProposals/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-specific-targets/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-versions-decimals/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs2-versions-strings/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs2/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-all-chrome-71/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-all/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-android/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-babel-polyfill/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-electron/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-es-chrome-71/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-es-proposals-chrome-71/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-es-proposals/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-es/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-force-all-transforms/stdout.txt
10	10	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-no-import/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-proposals-chrome-71/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-proposals/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-runtime-only-chrome-71/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-runtime-only/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-specific-entries-chrome-71/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-specific-entries/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-specific-targets/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-stable-chrome-71/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-stable-samsung-8.2/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-stable/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-stage-chrome-71/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-stage/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-versions-decimals/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-versions-strings-minor-3.0/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-versions-strings-minor-3.1/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-versions-strings/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-web-chrome-71/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3-web/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-corejs3/stdout.txt
10	10	packages/babel-preset-env/test/fixtures/debug/entry-no-corejs-no-import/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-no-corejs-shippedProposals/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-no-corejs-uglify/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/entry-no-corejs/stdout.txt
9	9	packages/babel-preset-env/test/fixtures/debug/plugins-only/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-1/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-2/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-chrome-71-1/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-chrome-71-2/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-none-1/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-none-2/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-proposals-1/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-proposals-2/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-proposals-chrome-71-1/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-proposals-chrome-71-2/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-shippedProposals-1/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-shippedProposals-2/stdout.txt
5	5	packages/babel-preset-env/test/fixtures/debug/usage-corejs2-with-import/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-1/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-2/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-chrome-71-1/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-chrome-71-2/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-none-1/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-none-2/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-proposals-1/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-proposals-2/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-proposals-chrome-71-1/stdout.txt
4	4	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-proposals-chrome-71-2/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-shippedProposals-1/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-shippedProposals-2/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-versions-strings-minor-3.0-1/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-versions-strings-minor-3.0-2/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-versions-strings-minor-3.1-1/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-versions-strings-minor-3.1-2/stdout.txt
5	5	packages/babel-preset-env/test/fixtures/debug/usage-corejs3-with-import/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-no-corejs-1/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-no-corejs-2/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-no-corejs-none-1/stdout.txt
11	11	packages/babel-preset-env/test/fixtures/debug/usage-no-corejs-none-2/stdout.txt
10	10	packages/babel-preset-env/test/fixtures/preset-options/safari-10_3-block-scoped/stdout.txt

Ref: #8138, #7586

@nicolo-ribaudo nicolo-ribaudo added PR: Bug Fix 🐛 A type of pull request used for our changelog categories PR: Fixes failing main pkg: preset-env i: regression labels Jan 16, 2020
"proposal-async-generator-functions": "Asynchronous Iterators",
"proposal-object-rest-spread": "object rest/spread properties",

// We want to apply this prior to unicode regex so that "." and "u"
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think the order of regex-related plugins still matters since they are essentially feature flags now, the regexpu-core should only run once for every applicable regex.

"proposal-optional-catch-binding": "optional catch binding",
"proposal-optional-chaining": "optional chaining operator (?.)",
"proposal-unicode-property-regex": "RegExp Unicode Property Escapes",
const es5 = {
Copy link
Contributor

Choose a reason for hiding this comment

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

Would be good to add a comment here if the key order will impact the plugin execution order of preset-env.

Copy link
Member

@existentialism existentialism left a comment

Choose a reason for hiding this comment

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

I coulda swore I had a comment about keeping order of the data, but I guess not ¯_(ツ)_/¯

@github-actions github-actions bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Apr 17, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
i: regression outdated A closed issue/PR that is archived due to age. Recommended to make a new issue pkg: preset-env PR: Bug Fix 🐛 A type of pull request used for our changelog categories PR: Fixes failing main
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants