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

bump Babel 8 to 8.0.0-alpha.12 #2961

Merged
merged 2 commits into from
Sep 17, 2024
Merged

bump Babel 8 to 8.0.0-alpha.12 #2961

merged 2 commits into from
Sep 17, 2024

Conversation

JLHwung
Copy link
Contributor

@JLHwung JLHwung commented Sep 12, 2024

The build is failing when bumping from 8.0.0-alpha.2 to 8.0.0-alpha.12.

The error is confusing since babel-loader 9 did not issue loadPartialConfig calls. I am still investigating the diff highlight regression, if you have bandwidth, please take a look.

To reproduce the build error, one can run yarn start:repl.

Copy link

netlify bot commented Sep 12, 2024

Deploy Preview for babel-next ready!

Name Link
🔨 Latest commit 7f736cc
🔍 Latest deploy log https://app.netlify.com/sites/babel-next/deploys/66e9b68d12c0d90008a94d02
😎 Deploy Preview https://deploy-preview-2961--babel-next.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Sep 12, 2024

Deploy Preview for babel failed.

Name Link
🔨 Latest commit 680d6fe
🔍 Latest deploy log https://app.netlify.com/sites/babel/deploys/66e3117217238d00082101ae

Copy link

netlify bot commented Sep 12, 2024

Deploy Preview for babel ready!

Name Link
🔨 Latest commit 7f736cc
🔍 Latest deploy log https://app.netlify.com/sites/babel/deploys/66e9b68dc8871e0008ac7daf
😎 Deploy Preview https://deploy-preview-2961--babel.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@liuxingbaoyu
Copy link
Member

This has been fixed in babel/babel-loader#1007.

@JLHwung
Copy link
Contributor Author

JLHwung commented Sep 12, 2024

@liuxingbaoyu Thank you! I guess the incorrect argument error was thrown asynchronously, otherwise it should have been caught in the catch clause.

@nicolo-ribaudo Could you take a look at babel/babel-loader#1037 and do a release from the 9.x branch?

@JLHwung
Copy link
Contributor Author

JLHwung commented Sep 17, 2024

CI is green again thanks to the babel-loader upgrade.

@JLHwung JLHwung merged commit 9291421 into babel:main Sep 17, 2024
4 checks passed
@JLHwung JLHwung deleted the bump-babel-8 branch September 17, 2024 18:18
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.

2 participants