-
-
Notifications
You must be signed in to change notification settings - Fork 5.8k
Do not enable default plugins in @babel/standalone
#17421
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
Conversation
|
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/59702 |
|
(The fact that it's 3AM and it took me two force-pushes to get a version of this that I believe is correct suggests I should probably at minimum manually test it tomorrow, if not figure out how to write a test for it, but right now I'm going to sleep.) |
Co-authored-by: Huáng Jùnliàng <[email protected]>
|
... failure might be spurious? I can't figure out what's going on in the log there. |
|
Yeah it's not related, I'm restarting that job. |
@babel/standalone
This is per #17419 (comment), but does not fully fix that issue.
As suggested I've made it a future change for Babel 8. I did not add any tests (and I wouldn't really know where to start), but I did run the existing tests for babel-standalone and they continue to pass. (I have no idea whether they even attempt to exercise any of this, though.)
I'm happy to tweak it for any local style things I missed, but I'd just as soon you edit it directly, if it's otherwise suitable.