-
-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
feat: move to read-package-json-fast #130
Conversation
This moves away from read-pkg to a much, much lighter package maintained by the npm team themselves (read-package-json-fast).
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/read-pkg@8.1.0 |
Right on, let's see if it breaks anything in CI. |
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #130 +/- ##
==========================================
- Coverage 96.82% 96.31% -0.52%
==========================================
Files 35 35
Lines 2080 2114 +34
==========================================
+ Hits 2014 2036 +22
- Misses 66 78 +12
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Looks good, will test locally and release tomorrow. |
Thanks so much for taking a look so soon! 🙏 |
No problem. I'll take all the easy, future facing improvements this can get! |
This moves away from read-pkg to a much, much lighter package maintained by the npm team themselves (read-package-json-fast).
Saves us ~90k of install size and the time spent fetching the 15-20 dependencies read-pkg has