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)

PR-URL: #3304
Credit: @NariyasuHeseri
Close: #3304
Reviewed-by: @wraithgar

PR-URL: #3307
Credit: @rethab
Close: #3307
Reviewed-by: @wraithgar

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

PR-URL: #3312
Credit: @isaacs
Close: #3312
Reviewed-by: @wraithgar

PR-URL: #3313
Credit: @spencerwilson
Close: #3313
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