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.16.0 #3322

Closed
wants to merge 18 commits into from
Closed

Release/v7.16.0 #3322

wants to merge 18 commits into from

Conversation

wraithgar
Copy link
Member

v7.16.0 (2021-05-27)

FEATURES

BUG FIXES

DEPENDENCIES

  • 7b56bfdf3 cacache@15.2.0:
    • feat: allow fully deleting indices
    • feat: add a validateEntry option to compact
    • chore: lint
    • chore: use standard npm style release scripts
  • dbbc151a3 npm-audit-report@2.1.5:
  • 5b2604507 chore(package-lock): update devDependencies (@Gar)

DOCUMENTATION

AUTOMATION

KevinFCormier and others added 18 commits May 24, 2021 11:33
I think this change was incorrect: #2024

The point of this example is that the same script is being used for two different stages (`install` and `post-install`) so it would be a good idea to look at the `npm_lifecycle_event` environment variable inside this script to determine which stage is being run.

PR-URL: #3282
Credit: @KevinFCormier
Close: #3282
Reviewed-by: @wraithgar
Change the output in `npm ls` for symlink dependencies (including
workspaces) to show the relative path from the project root, rather than
the absolute path of the link target.

This makes the output much less noisy and more ergonomic when many
workspaces and link dependencies are in use, especially when paths are
long.

It is arguable that this output might be slightly misleading, since the
_actual_ workspace symlink from `${root}/node_modules/b` to
`${root}/packages/b` has a link value of `../packages/b`, not just
`packages/b`.  (Or on Windows, always the full absolute path, because
junctions.)  Thus, `npm ls b` will not show the same output as `ls -l
node_modules/b`.

However, `npm ls` shows the logical tree, not the physical tree, so
presenting the user with a path that they can use and interpret is more
important than presenting them with the strictly accurate details of
their filesystem.

PR-URL: #3272
Credit: @isaacs
Close: #3272
Reviewed-by: @darcyclarke
 * fix(exit-code): account for null auditLevel default (#46)
When running 'npm audit', we properly exited correctly with the
appropriate exitCode based on the audit level config and the report
results.

However, when going through the reifyFinish() function (as we do for
'npm audit fix'), we were not setting that properly if the auditLevel
was not set.

Furthermore, if the auditLevel WAS set, we were setting the exit code to
non-zero for *other* reify commands (install, update, etc.), where the
audit information should be strictly advisory.

When --json and --loglevel=silent were set, the exitCode was never being
set properly.

This fixes all these problems.

PR-URL: #3311
Credit: @isaacs
Close: #3311
Reviewed-by: @wraithgar
Allow users to publish all workspaces with `npm publish --ws` while also
skipping any workspace that might have been intentionally marked as
private, using `"private": true` in its package.json file.

Fixes: #3268

PR-URL: #3285
Credit: @ruyadorno
Close: #3285
Reviewed-by: @wraithgar
  * devDependency used for building html docs
0.9.0 does not work in node 10.1 so our CI fails
 * feat: allow fully deleting indices
 * feat: add a validateEntry option to compact
 * chore: lint
 * chore: use standard npm style release scripts
Moves workflow to create NPM CLI dependency pull request from npm/node.
Adds input to workflow dispatch trigger

PR-URL: #3294
Credit: @gimli01
Close: #3294
Reviewed-by: @darcyclarke
@wraithgar wraithgar requested a review from a team as a code owner May 27, 2021 20:04
@wraithgar wraithgar closed this May 27, 2021
@wraithgar wraithgar deleted the release/v7.16.0 branch May 27, 2021 20:09
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.

8 participants