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/v8.4.1 #4369

Merged
merged 18 commits into from
Feb 3, 2022
Merged

Release/v8.4.1 #4369

merged 18 commits into from
Feb 3, 2022

Conversation

ruyadorno
Copy link
Contributor

v8.4.1 (2022-02-03)

Bug Fixes

Documentation

Dependencies

wraithgar and others added 18 commits February 1, 2022 14:26
Most of these module use npm-registry-fetch under the hood, which will
log things like the `npm-notice` header if seen.  Currently we aren't
passing in a logger to them, which means that log message is never seen,
among any other logged messages those modules may need to make.

I added tests where I could. Some tests were in a state where the entire
libnpm* module was an empty mocked function, so asserting that it got
passed a `log` attribute was onerous.
This has been possible since this commit in `npm-bundled`:
npm/npm-bundled@101a94d
Make sure to validate any lock file (either package-lock.json or
npm-shrinkwrap.json) against the current install. This will properly
throw an error in case any of the dependencies being installed don't
match the dependencies that are currently listed in the lock file.

Fixes: #2701
Fixes: #3947
`npm ci` should never be affected by the `package-lock` config.

Fixes: #4185
Fixes `npm outdated` to properly parse and display info on aliased
packages.

Fixes: #2800
@ruyadorno ruyadorno requested a review from a team as a code owner February 3, 2022 20:04
@npm-robot
Copy link
Contributor

found 20 benchmarks with statistically significant performance improvements

  • app-large: clean, lock-only, cache-only, cache-only:peer-deps, modules-only, no-lock, no-cache, no-modules, no-clean, no-clean:audit
  • app-medium: clean, lock-only, cache-only, cache-only:peer-deps, modules-only, no-lock, no-cache, no-modules, no-clean, no-clean:audit
timing results
app-large clean lock-only cache-only cache-only
peer-deps
modules-only no-lock no-cache no-modules no-clean no-clean
audit
npm@8 40.087 ±0.46 26.368 ±0.15 15.092 ±0.20 18.075 ±0.64 2.666 ±0.02 2.676 ±0.01 2.169 ±0.01 10.522 ±0.11 2.162 ±0.01 3.007 ±0.03
#4369 0.378 ±0.00 0.370 ±0.00 0.375 ±0.00 0.375 ±0.00 0.378 ±0.01 0.373 ±0.00 0.372 ±0.00 0.374 ±0.00 0.376 ±0.01 0.372 ±0.00
app-medium clean lock-only cache-only cache-only
peer-deps
modules-only no-lock no-cache no-modules no-clean no-clean
audit
npm@8 30.757 ±0.40 20.679 ±0.14 11.756 ±0.03 12.705 ±0.43 2.445 ±0.02 2.403 ±0.01 2.158 ±0.01 7.796 ±0.04 2.026 ±0.02 2.738 ±0.01
#4369 0.373 ±0.00 0.374 ±0.00 0.379 ±0.01 0.374 ±0.00 0.373 ±0.00 0.372 ±0.00 0.389 ±0.02 0.372 ±0.00 0.376 ±0.01 0.374 ±0.00

@ruyadorno ruyadorno merged commit 892b66e into latest Feb 3, 2022
@ruyadorno ruyadorno deleted the release/v8.4.1 branch February 3, 2022 20:34
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