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

plugin-transform-modules-commonjs: export binding not updated by 'for...in' and 'for...of' #10941

Open
ajafff opened this issue Dec 29, 2019 · 1 comment

Comments

@ajafff
Copy link
Contributor

@ajafff ajafff commented Dec 29, 2019

Bug Report

  • I would like to work on a fix!

Current Behavior
If an exported binding is used as LHS in a ForOfStatement or ForInStatement, only the local variable is updated. The exported binding stays the same.

Input Code

export let foo = 'initial';

for (foo in {prop: 1}) {}

for (foo of ['element']) {}

for ({foo} of [{foo: 'destructured'}]) {}

Expected behavior/code
exports.foo should be updated every time foo is assigned a new value.
I tested the code without transpilation in Node.js and it works as expected.

@babel-bot

This comment has been minimized.

Copy link
Collaborator

@babel-bot babel-bot commented Dec 29, 2019

Hey @ajafff! We really appreciate you taking the time to report an issue. The collaborators on this project attempt to help as many people as possible, but we're a limited number of volunteers, so it's possible this won't be addressed swiftly.

If you need any help, or just have general Babel or JavaScript questions, we have a vibrant Slack community that typically always has someone willing to help. You can sign-up here for an invite."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.