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
chore: Helper changes for Babel 8 #16209
Conversation
Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/56120/ |
"alongside 'file.availableHelper()'.", | ||
); | ||
if (!process.env.BABEL_8_BREAKING) { | ||
if (key === "helpersNamespace") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for reminding!
This is a special case. :)
Line 499 in 0085301
function pluginToggleBooleanFlag({ types: t }, { name, value }) { |
We use a custom plugin to remove it on release.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, since #15961 the custom plugin should support doing
if (!process.env.BABEL_8_BREAKING && key === "helpersNamespace") {
:)
56d64f0
to
e94edf0
Compare
Fixes #1, Fixes #2
The only change is that we no longer access
node._loc
.To be honest, just removing these two methods is fine for me too.