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
Fix update expression for exported bigints #14341
Conversation
@@ -20,73 +16,6 @@ export default function simplifyAccess(path: NodePath, bindingNames) { | |||
} | |||
|
|||
const simpleAssignmentVisitor = { | |||
UpdateExpression: { |
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.
If someone accidentally updates @babel/helper-simple-access
without also updating @babel/helper-module-transforms
(this is a valid thing to do, according to the semver dependency versions), I fear that this will break their current code.
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.
Yes.
What can I do? Should I bump the version of the package?
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.
No, because we keep all the versions synchronized (and it would require a major bump).
It's annoying, but a solution until Babel 8 is to pass a includeUpdateExpression: boolean = true
option to simplifyAccess
, and only skip the UpdateExpression
visitor when that option is set to false
.
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.
for the new tests
@babel/helper-simple-access
changed