-
-
Notifications
You must be signed in to change notification settings - Fork 598
refactor: Remove @babel/runtime
from production dependencies
#1777
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
Conversation
Thanks for opening this pull request!
|
Codecov ReportBase: 99.90% // Head: 99.90% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## alpha #1777 +/- ##
=======================================
Coverage 99.90% 99.90%
=======================================
Files 61 61
Lines 6103 6103
Branches 1482 1482
=======================================
Hits 6097 6097
Misses 6 6 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
This PR is to show my support for the core-js community. |
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.
Looks good!
🎉 This change has been released in version 4.1.0-beta.1 |
🎉 This change has been released in version 4.1.0-alpha.1 |
🎉 This change has been released in version 4.1.0 |
Pull Request
Issue
We don't need both
@babel/runtime
and@babel/runtime-corejs3
.@babel/runtime
is now a dev dependency since it included in@babel/preset-env
.@babel/runtime
was left in during the babel 7 migration.https://babeljs.io/docs/en/v7-migration#helpers--polyfilling-from-core-js
Using the longhand version of
@babel/transform-runtime
for clarityhttps://babeljs.io/docs/en/v7-migration#package-renames