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

Release/v7.4.1 #2494

Merged
merged 22 commits into from
Jan 14, 2021
Merged

Release/v7.4.1 #2494

merged 22 commits into from
Jan 14, 2021

Conversation

wraithgar
Copy link
Member

v7.4.1 (2021-01-14)

BUG FIXES

  • 23df96d33 #2486 npm link no longer deletes entire project when global prefix is a symlink (@nlf)

DOCUMENTATION

DEPENDENCIES

wraithgar and others added 22 commits January 14, 2021 13:15
Adds an example, and a note about how this differs than node's default
behavior

PR-URL: #2459
Credit: @wraithgar
Close: #2459
Reviewed-by: @darcyclarke
Adds an example and outlines differences with `npm start`

PR-URL: #2460
Credit: @wraithgar
Close: #2460
Reviewed-by: @darcyclarke
Adds an example and standardizes the language w/ start and stop

PR-URL: #2462
Credit: @wraithgar
Close: #2462
Reviewed-by: @darcyclarke
Adds an example and slightly changes wording

PR-URL: #2463
Credit: @wraithgar
Close: #2463
Reviewed-by: @darcyclarke
Lots of flags removed, clarification on what `--no-save` does,
consolidated examples into a separate section
Co-authored-by: Jordan Harband <ljharb@gmail.com>
Co-authored-by: Darcy Clarke <darcy@darcyclarke.me>
Co-authored-by: Darcy Clarke <darcy@darcyclarke.me>

PR-URL: #2464
Credit: @wraithgar
Close: #2464
Reviewed-by: @darcyclarke
Mostly grammar fixes, also adds not about needing to
be logged in, as per comment in #1880

PR-URL: #2474
Credit: @wraithgar
Close: #2474
Reviewed-by: @darcyclarke
grammar fixes, notes on authentication tokens, match usage
output with readme

PR-URL: #2482
Credit: @wraithgar
Close: #2482
Reviewed-by: @darcyclarke
correctly identify already linked packages when global prefix is a symlink

PR-URL: #2486
Credit: @nlf
Close: #2486
Reviewed-by: @wraithgar
In which the "note on caching" is finally populated after
all these years

PR-URL: #2487
Credit: @wraithgar
Close: #2487
Reviewed-by: @darcyclarke
@wraithgar wraithgar requested a review from a team as a code owner January 14, 2021 22:05
@isaacs
Copy link
Contributor

isaacs commented Jan 14, 2021

It might be worth mentioning in the changelog entry for arborist that npm audit got significantly faster.

@wraithgar wraithgar merged commit 653769d into latest Jan 14, 2021
@npm-deploy-user
Copy link

found 3 benchmarks with statistically significant performance regressions

  • app-large: no-clean:audit
  • app-medium: cache-only:legacy-peer-deps, modules-only
timing results
app-large clean lock-only cache-only cache-only
legacy-peer-deps
modules-only no-lock no-cache no-modules no-clean no-clean
audit
npm@7 45.057 ±1.13 28.133 ±0.32 29.078 ±0.56 25.714 ±0.42 2.179 ±0.10 2.126 ±0.01 1.514 ±0.02 11.000 ±0.09 1.525 ±0.00 3.253 ±0.04
#2494 46.627 ±0.45 29.063 ±0.02 29.590 ±0.47 26.538 ±0.95 2.258 ±0.01 2.330 ±0.01 1.642 ±0.04 11.620 ±0.08 1.601 ±0.00 3.857 ±0.48
app-medium clean lock-only cache-only cache-only
legacy-peer-deps
modules-only no-lock no-cache no-modules no-clean no-clean
audit
npm@7 31.732 ±0.30 19.641 ±0.27 19.212 ±0.53 18.012 ±0.77 2.019 ±0.01 2.024 ±0.04 1.406 ±0.00 7.299 ±0.09 1.433 ±0.03 2.641 ±0.02
#2494 33.804 ±0.30 20.615 ±0.21 19.509 ±0.66 20.140 ±0.54 2.260 ±0.02 2.194 ±0.03 1.517 ±0.02 7.555 ±0.02 1.516 ±0.01 3.149 ±0.61

@nlf nlf deleted the release/v7.4.1 branch March 28, 2022 20:50
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.

4 participants