-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
🏗🚮 Clean up package.json
by removing unused / adding missing deps
#21105
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.
I would still prefer to split the runtime-test/index.js
into a separate PR, since it looks like an unrelated fix (and it's easier to revert and/or cherry-pick smaller PRs when needed) but it's up to you
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.
LGTM
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.
LGTM
I ran
npm-check
anddepcheck
to generate a list of unused and missing dependencies inpackage.json
. This PR adds / removes them as appropriate.Unfortunately, it's not easy to add an always-on check that runs
npm-check
anddepcheck
for all PRs because the tools report several false positives for dependencies that are in fact used in build / test tools (where there is norequire
orimport
statement that mentions the dependency).