Age | Commit message (Collapse) | Author |
|
|
|
PR-URL: https://github.com/npm/cli/pull/4144
Credit: @nlf
Close: #4144
Reviewed-by: @wraithgar
|
|
* Fixes object property assignment bug in resolving package-locks with
conflicts
PR-URL: https://github.com/npm/cli/pull/4141
Credit: @wraithgar
Close: #4141
Reviewed-by: @fritzy
|
|
PR-URL: https://github.com/npm/cli/pull/4116
Credit: @nlf
Close: #4116
Reviewed-by: @wraithgar
|
|
|
|
PR-URL: https://github.com/npm/cli/pull/4109
Credit: @wraithgar
Close: #4109
Reviewed-by: @lukekarrys
|
|
|
|
|
|
This decouples the log file writing from the terminal logging. We now
open an append only file at the start of the process and stream logs to
it. We still only display the log file message in timing mode or if
there is an error, but the file is still written regardless.
All logging now goes through `proc-log` and this is the first step to
removing `npmlog`. For now `npmlog` is still used for the terminal
logging but with a shim in front of it to make it easier to test and
use in conjunction with `proc-log`. Ref: npm/statusboard#366
This also refactors many of the tests to always use an explicit
`t.testdir` for their cache since the file is opened on each `new Npm()`.
Tests are also refactored to use more of `MockNpm` with behavior to
add config items and load `npm` if necessary. A new fixture `mockGlobals`
was also added to make much of this more ergonomic. Ref: npm/statusboard#410
Closes npm/statusboard#411
Closes npm/statusboard#367
PR-URL: https://github.com/npm/cli/pull/4062
Credit: @lukekarrys
Close: #4062
Reviewed-by: @wraithgar
|
|
PR-URL: https://github.com/npm/cli/pull/4078
Credit: @wraithgar
Close: #4078
Reviewed-by: @lukekarrys
|
|
* fix: always load localPrefix
PR-URL: https://github.com/npm/cli/pull/4102
Credit: @wraithgar
Close: #4102
Reviewed-by: @lukekarrys
|
|
|
|
PR-URL: https://github.com/npm/cli/pull/4065
Credit: @fritzy
Close: #4065
Reviewed-by: @lukekarrys
|
|
* feat: support vs2022
* feat: build with config.gypi from node headers
PR-URL: https://github.com/npm/cli/pull/4045
Credit: @wraithgar
Close: #4045
Reviewed-by: @lukekarrys
|
|
|
|
PR-URL: https://github.com/npm/cli/pull/4048
Credit: @wraithgar
Close: #4048
Reviewed-by: @fritzy
|
|
PR-URL: https://github.com/npm/cli/pull/4049
Credit: @wraithgar
Close: #4049
Reviewed-by: @fritzy
|
|
|
|
PR-URL: https://github.com/npm/cli/pull/4050
Credit: @fritzy
Close: #4050
Reviewed-by: @wraithgar
PR-URL: https://github.com/npm/cli/pull/4050
Credit: @fritzy
Close: #4050
Reviewed-by: @wraithgar
|
|
* Update to eslint@8 and and @npmcli/eslint-config@2.0.0
* Remove eslint-plugin-node.
Also remove an unused script that was failing linting. We don't use the
update-dist-tags script anymore as part of our release process.
PR-URL: https://github.com/npm/cli/pull/4016
Credit: @wraithgar
Close: #4016
Reviewed-by: @lukekarrys
|
|
This brings us in line with the rest of the linting rules we are wanting
to use on the npm cli repos.
There were several hundred over-length lines and instead of editing them
all by hand I piped the failing file through `prettier` and back through
`eslint` just to save some time and energy. This means there may be
some quirks in the linting those files have, but we can fix those if we
see them and they bother us.
Other than that there were about 50 lines that are legitimately
over-length, all are now explicitly overridden. Many are tests that
could be snapshots.
PR-URL: https://github.com/npm/cli/pull/3995
Credit: @wraithgar
Close: #3995
Reviewed-by: @lukekarrys
|
|
|
|
* fix: dont load project configs in global mode
PR-URL: https://github.com/npm/cli/pull/3996
Credit: @nlf
Close: #3996
Reviewed-by: @wraithgar
PR-URL: https://github.com/npm/cli/pull/3996
Credit: @nlf
Close: #3996
Reviewed-by: @wraithgar
|
|
PR-URL: https://github.com/npm/cli/pull/3990
Credit: @lukekarrys
Close: #3990
Reviewed-by: @wraithgar
|
|
|
|
PR-URL: https://github.com/npm/cli/pull/3953
Credit: @lukekarrys
Close: #3953
Reviewed-by: @nlf
|
|
|
|
* fix: skip peer conflict check if there is a current node
PR-URL: https://github.com/npm/cli/pull/3921
Credit: @lukekarrys
Close: #3921
Reviewed-by: @wraithgar
|
|
* feat(gyp): update gyp to v0.10.0
PR-URL: https://github.com/npm/cli/pull/3913
Credit: @wraithgar
Close: #3913
Reviewed-by: @lukekarrys
|
|
|
|
PR-URL: https://github.com/npm/cli/pull/3893
Credit: @nlf
Close: #3893
Reviewed-by: @nlf
|
|
|
|
PR-URL: https://github.com/npm/cli/pull/3889
Credit: @wraithgar
Close: #3889
Reviewed-by: @nlf
|
|
* fix: preserve git+ssh url for non-hosted repos
* deps: update npm-packlist@3.0.0
* fix: no longer include ignored bundled link deps
PR-URL: https://github.com/npm/cli/pull/3883
Credit: @wraithgar
Close: #3883
Reviewed-by: @nlf
|
|
Bumped patch versions of libnpmexec and libnpmfund to dedupe
|
|
|
|
PR-URL: https://github.com/npm/cli/pull/3813
Credit: @lukekarrys
Close: #3813
Reviewed-by: @wraithgar
|
|
BREAKING CHANGE:
- Drop official support for node versions less than v12.
- Drop support for `require('npm')`
- Update a few subdependencies that dropped node10 support, and brought
in the latest node-gyp
PR-URL: https://github.com/npm/cli/pull/3762
Credit: @wraithgar
Close: #3762
Reviewed-by: @fritzy
|
|
|
|
|
|
This reverts commit 8349c3c1557ac52973ad08c10db492e3a5a30204.
PR-URL: https://github.com/npm/cli/pull/3817
Credit: @lukekarrys
Close: #3817
Reviewed-by: @wraithgar
|
|
This reverts commit f17dfa0ced7d8df9bb7baf378bb20d33175c8e8b.
|
|
Adds a new config item that includes the workspace root when running
non-arborist commands (i.e. repo, version, publish). Arborist will need
to be udpated to look for this flag to change its behavior to include
the workspace root for its functions.
This also changes --workspaces to a trinary, so that setting it to false
will explicitly exclude workspaces altogether. This is also going to
require an arborist change so that it ignores workspaces altogether.
Co-author: @fritzy
PR-URL: https://github.com/npm/cli/pull/3816
Credit: @isaacs
Close: #3816
Reviewed-by: @wraithgar
|
|
* includeWorkspaceRoot support
* workspacesEnabled=false support
PR-URL: https://github.com/npm/cli/pull/3815
Credit: @wraithgar
Close: #3815
Reviewed-by: @isaacs
|
|
The npm/cli form of https://github.com/npm/arborist/pull/324
Required adding options support to package used for this.
PR-URL: https://github.com/npm/cli/pull/3809
Credit: @isaacs
Close: #3809
Reviewed-by: @wraithgar
|
|
* fix: avoid infinite loops in peer dep replacements
* fix: use Intl.Collator for string sorting when available
* feat(vuln): expose isDirect
|
|
|
|
|
|
* feat: allow passing tls connection options
|
|
|