Age | Commit message (Collapse) | Author |
|
PR-URL: https://github.com/npm/cli/pull/3761
Credit: @wraithgar
Close: #3761
Reviewed-by: @lukekarrys
|
|
If the package being installed is a url it needs to be cleaned before
logging so passwords aren't potentially logged.
PR-URL: https://github.com/npm/cli/pull/3732
Credit: @wraithgar
Close: #3732
Reviewed-by: @nlf
|
|
The attribute is here, er.file was used in error on another lib, the lib
got changed, and this code wasn't fixed.
PR-URL: https://github.com/npm/cli/pull/3661
Credit: @wraithgar
Close: #3661
Reviewed-by: @fritzy
|
|
The config value for `color` should never be directly read by npm, or
its submodules. The derived value `npm.color` or
`npm.flatOptions.color` is what we want to use.
This PR consolidates the use of these values, makes sure there is only
one place the value is actually calculated, and stops relying on
duplicated code in the setup-log.js file for setting one of them.
PR-URL: https://github.com/npm/cli/pull/3563
Credit: @wraithgar
Close: #3563
Reviewed-by: @lukekarrys
|
|
PR-URL: https://github.com/npm/cli/pull/3498
Credit: @nlf
Close: #3498
Reviewed-by: @wraithgar
|
|
No more requiring npm as a singleton. This will now allow us to move
forward with the other refactoring we need to always use the npm object
itself in tests, not a mocked one.
PR-URL: https://github.com/npm/cli/pull/3388
Credit: @wraithgar
Close: #3388
Reviewed-by: @ruyadorno
|
|
PR-URL: https://github.com/npm/cli/pull/3069
Credit: @isaacs
Close: #3069
Reviewed-by: @ruyadorno, @nlf
|
|
We have a lot of tests that were not handling errors in callbacks, not
defining variables, defining variables that weren't used, and just
generally not formatted like the rest of npm.
Hazard of moving fast. Thankfully, machines can help.
|
|
|
|
* Fix EBADPLATFORM error message
Error format evolved out from under message generation's
expectations.
* Fix formatting
|
|
When peerDependencies conflict, Arborist is now providing details of the
nodes and their reasons for inclusion on the Error object, including
whether or not this resolution error could be overridden using the
--force flag.
Print this data out in a minimal way as a warning if we override an
ERESOLVE forcefully. When the ERESOLVE causes the install to fail,
print a somewhat longer message, and save a MUCH longer full report to
the cache folder.
PR-URL: https://github.com/npm/cli/pull/1761
Credit: @isaacs
Close: #1761
Reviewed-by: @darcyclarke, @ruyadorno
|
|
Also, get Windows up to full coverage.
PR-URL: https://github.com/npm/cli/pull/1598
Credit: @isaacs
Close: #1598
Reviewed-by: @ruyadorno
|
|
PR-URL: https://github.com/npm/cli/pull/1509
Credit: @isaacs
Close: #1509
Reviewed-by: @ruyadorno
|
|
|