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

Keep only relevant npm WordPress packages #1466

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

Conversation

kadimi
Copy link
Contributor

@kadimi kadimi commented Sep 20, 2020

Changes proposed in this Pull Request:

The @wordpress/scripts npm package offers 13 scripts but only 2 are used by _s. I thought that adding only those two scripts to the project instead of the whole family would improve it and it certainly did. Here is a comparison:

npm install Before the Change

added 1884 packages from 774 contributors and audited 1887 packages in 53.306s
# ...
found 2 vulnerabilities

npm install After the Change

added 604 packages from 373 contributors and audited 604 packages in 15.74s
# ...
found 0 vulnerabilities

Comparison

Before After % Improvement
Packages 1884 604 312%
Vulnerabilities 2 0 ∞%
Time 53.306s 15.74s 339%

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

Successfully merging this pull request may close these issues.

1 participant