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

Disable use @babel/runtime-corejs2 #180

Merged
merged 1 commit into from
Feb 21, 2020
Merged

Disable use @babel/runtime-corejs2 #180

merged 1 commit into from
Feb 21, 2020

Conversation

ykzts
Copy link
Contributor

@ykzts ykzts commented Feb 19, 2020

Description of Change(s):

Change next/babel options and don't use @babel/runtime-corejs2. This will allow you to build Next.js v9.2.3 without any problems.

Resolve #178

@ykzts ykzts requested a review from garmeeh February 21, 2020 15:05
@@ -1,5 +1,9 @@
{
presets: ['next/babel', '@babel/typescript', 'minify'],
presets: [
['next/babel', { 'transform-runtime': { corejs: false } }],
Copy link
Owner

Choose a reason for hiding this comment

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

Thanks @ykzts. Sorry I haven't had a chance to read up on this just yet. Would you consider this a breaking change? Will it affect older browsers for example on the current version of next?

I will look at getting this released this evening after work.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The only difference is whether or not core-js is used. It does not affect older browsers or previous versions of Next.js.

@garmeeh
Copy link
Owner

garmeeh commented Feb 21, 2020

@all-contributors please add @ykzts code

@allcontributors
Copy link
Contributor

@garmeeh

I've put up a pull request to add @ykzts! 🎉

Copy link
Owner

@garmeeh garmeeh left a comment

Choose a reason for hiding this comment

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

Thanks for the update 😄

@garmeeh garmeeh merged commit 95ca537 into garmeeh:master Feb 21, 2020
@ykzts ykzts deleted the disable-corejs branch February 22, 2020 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

experimental flag polyfillsOptimization does not work with this module
2 participants