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

chore: Reduce build size by importing core-js #641

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lutangar
Copy link
Member

@lutangar lutangar commented Jun 4, 2020

Something seems wrong with our config, just with this change it seems the size of included core-js functions is reduced 4 times...

Something seems wrong with our config, just with this change it seems the size of included core-js functions is reduced 4 times...
@lutangar lutangar requested a review from JalilArfaoui June 4, 2020 08:24
@lutangar lutangar self-assigned this Jun 4, 2020
@lutangar
Copy link
Member Author

lutangar commented Jun 4, 2020

I forgot to create this PR a while back. But I think we are just missing the preset-env bit here, this would do the job for us with the right config.

https://babeljs.io/docs/en/babel-preset-env#corejs

@JalilArfaoui
Copy link
Member

@lutangar I don’t see how it reduces the build size, since we’re still importing it another way … Did you test this with yarn analyze  ?

@lutangar
Copy link
Member Author

Did you test this with yarn analyze

Yes I did, but things might have change since then. Maybe it would look less like webpack voodoo magic today....

@JalilArfaoui
Copy link
Member

@lutangar I’m still not convinced this makes any difference … Could you try to rebase and check ? Or do we close it ? 

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

Successfully merging this pull request may close these issues.

None yet

2 participants