-
Notifications
You must be signed in to change notification settings - Fork 5
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
'loose' mode configuration must be the same for both @babel/plugin-proposal-class-properties and @babel/plugin-proposal-private-methods #11
Comments
Are you able to share your project @Mrtenz? |
Here's a minimal repo where I'm able to reproduce: Simply running |
I use the same preset that Gatsby uses to transpile their own files, with an absolute runtime path injected for AFAIK, What I don't understand is why it is throwing now, because it's been like that for a while, and running successfully. Maybe babel changed the way they process those options at some point recently. I will take a look and see what I can come up with. |
This may be related: babel/babel#11622 |
That's exactly why. As soon as I add that option to |
This is what's new; merged in 13 hours ago. Pretty sure that's what caused it to start breaking.. they added some options constraints between a few plugins, those two included. The semver for |
PR is opened gatsbyjs/gatsby#24640 |
Thanks for bringing that to my attention, by the way 🙏 |
Adding a selective dependency resolution for |
This works for me: "resolutions": {
"@babel/helper-create-class-features-plugin": "7.9.6"
} |
Can you guys please try |
|
v0.2.3 is published 🚀 |
Using this plugin with Babel as transpiler results in the following error:
I'm not using a custom Babel config, so I'm assuming this is an issue with this plugin. Setting
babel
tofalse
andtsNode
totrue
resolves it, but I'd like to use Babel if possible.The text was updated successfully, but these errors were encountered: