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(babel-preset-gatsby-package): Add corejs to config #17723
fix(babel-preset-gatsby-package): Add corejs to config #17723
Conversation
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.
We're currently using version 2 everywhere as we don't want to upgrade yet. Core-js 3 has fixed some browser targets which sadly make everything bigger.
You'll also want to add core-js as a dependency.
Co-Authored-By: Ward Peeters <ward@coding-tech.com>
Oh sorry, just committed the changes, thanks for explaining it! I think now is ready :) |
Co-Authored-By: Sidhartha Chatterjee <me@sidharthachatterjee.com>
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.
Thank you so much! 🙌
Thanks to you guys! Super glad to contribute 😄 |
Holy buckets, @gonzarodriguezt — we just merged your PR to Gatsby! 💪💜 Gatsby is built by awesome people like you. Let us say “thanks” in two ways:
If there’s anything we can do to help, please don’t hesitate to reach out to us: tweet at @gatsbyjs and we’ll come a-runnin’. Thanks again! |
Cheers! |
@baerrach sorry for late reply - I'm currently publishing, so it should be out in few minutes, will post new comment when it's done |
|
Look at that lack of warnings! Thanks. |
* Add corejs to babel-preset-gatsby-package config * Update packages/babel-preset-gatsby-package/index.js Co-Authored-By: Ward Peeters <ward@coding-tech.com> * Add corejs as a dependency * Update snapshot * Update packages/babel-preset-gatsby-package/package.json Co-Authored-By: Sidhartha Chatterjee <me@sidharthachatterjee.com>
Description
This is a warning in development about a missing
corejs
config in babel, the issue below tracks down the source of the problem, and this fix quiets it down, thanks!Edit: I specified the version 3 as it seems to be the recommended version on the docs (here)
Related Issues
Fixed #17706