Skip to content

[babel 8] Enable preset-env bugfixes by default#13866

Merged
nicolo-ribaudo merged 4 commits into
babel:mainfrom
JLHwung:enable-bugfixes-by-default
Oct 24, 2021
Merged

[babel 8] Enable preset-env bugfixes by default#13866
nicolo-ribaudo merged 4 commits into
babel:mainfrom
JLHwung:enable-bugfixes-by-default

Conversation

@JLHwung

@JLHwung JLHwung commented Oct 20, 2021

Copy link
Copy Markdown
Contributor
Q                       A
Major: Breaking Change? Y, under BABEL_8_BREAKING flag
Tests Added + Pass? Yes
Documentation PR Link babel/website#2776
Any Dependency Changes?
License MIT

In this PR we enable the bugfixes preset-env option by default in Babel 8. Most updated test fixtures already have related Babel 7 tests so we don't need to add new Babel 7 test fixtures.

Comment thread .yarn/patches/@babel__preset-modules.patch Outdated
@codesandbox-ci

codesandbox-ci Bot commented Oct 20, 2021

Copy link
Copy Markdown

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 5439a0b:

Sandbox Source
babel-repl-custom-plugin Configuration
babel-plugin-multi-config Configuration

{
"targets": [
"Chrome >= 60",
"Chrome >= 54",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The issue #9935 happens when async function is transpiled to generator functions but class is not transformed. Now that the bugfixes lowers the Safari's support of async functions, we will not see async function transpiled at all in the old targets settings. Here I lower the Chromium version to the latest without native async function support so we are still testing the regression 9935.

@JLHwung JLHwung added this to the Babel 8.0 milestone Oct 20, 2021
@JLHwung
JLHwung force-pushed the enable-bugfixes-by-default branch from 3fc3498 to 5439a0b Compare October 21, 2021 17:36
@JLHwung

JLHwung commented Oct 21, 2021

Copy link
Copy Markdown
Contributor Author

The CI error is unrelated, it is caused by yarnpkg/berry#3597

@nicolo-ribaudo nicolo-ribaudo changed the title Enable preset-env bugfixes by default [babel 8] Enable preset-env bugfixes by default Oct 24, 2021
@nicolo-ribaudo
nicolo-ribaudo merged commit e45d86c into babel:main Oct 24, 2021
@nicolo-ribaudo
nicolo-ribaudo deleted the enable-bugfixes-by-default branch October 24, 2021 13:17
@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 Jan 24, 2022
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jan 24, 2022
@JLHwung JLHwung added PR: Breaking Change 💥 A type of pull request used for our changelog categories for next major release and removed babel 8 labels Aug 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: Breaking Change 💥 A type of pull request used for our changelog categories for next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants