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

Conversion of large map using _defineProperty #11027

Open
ralphsol opened this issue Jan 17, 2020 · 2 comments
Open

Conversion of large map using _defineProperty #11027

ralphsol opened this issue Jan 17, 2020 · 2 comments

Comments

@ralphsol
Copy link

@ralphsol ralphsol commented Jan 17, 2020

For a given js file, which contains only a large map such as below -
-----------------------‐---------f.js-----------------------‐-------------
module.exports = {"A":"1", "B":"2", "C":"3", ...}
-----------------------‐--------------------------------‐-------------------

And with preset as 'es2015', Babel keeps the file unchanged till some occurrences. But beyond a count, it replaces the latter instances of the map with _defineProperty(_module$exports, "Z", "26");
How can this threshold be adjusted?

@babel-bot

This comment has been minimized.

Copy link
Collaborator

@babel-bot babel-bot commented Jan 17, 2020

Hey @ralphsol! 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."

@nicolo-ribaudo

This comment has been minimized.

Copy link
Member

@nicolo-ribaudo nicolo-ribaudo commented Jan 17, 2020

Can you provide a reproducible example? The transformation fo _defineProperty shouldn't happen if you are already using a normal object.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.