Age | Commit message (Collapse) | Author |
|
|
|
|
|
|
|
This is not implemented in npm@7
PR-URL: https://github.com/npm/cli/pull/3486
Credit: @wraithgar
Close: #3486
Reviewed-by: @nlf
|
|
This is no longer possible, as per [rfc
21](https://github.com/npm/rfcs/blob/latest/implemented/0021-reduce-lifecycle-script-environment.md)
PR-URL: https://github.com/npm/cli/pull/3485
Credit: @wraithgar
Close: #3485
Reviewed-by: @isaacs
|
|
Closes https://github.com/npm/cli/issues/1623
PR-URL: https://github.com/npm/cli/pull/3483
Credit: @wraithgar
Close: #3483
Reviewed-by: @isaacs
|
|
Setting a deprecation of an empty string is the way to un-deprecate a
package, this was accidentally broken in a past refactoring, and is now
re-added with a test to ensure it is allowed.
PR-URL: https://github.com/npm/cli/pull/3484
Credit: @wraithgar
Close: #3484
Reviewed-by: @isaacs
|
|
There is no need to make the time divided by 1000 because of the time unit 'ms'.
Currently a typical command and response:
PS C:\projects> npm ping
npm notice PING http://registry.npmjs.org/
npm notice PONG 0.752ms
PR-URL: https://github.com/npm/cli/pull/3474
Credit: @aluneed
Close: #3474
Reviewed-by: @wraithgar
|
|
This moves the logging of the exit code to before the logfile is
written, when the exit handler was not called. This will ensure that
the code shows up in the debug logs.
PR-URL: https://github.com/npm/cli/pull/3469
Credit: @wraithgar
Close: #3469
Reviewed-by: @isaacs
|
|
|
|
|
|
|
|
* bin: allow turning off timer display with --timers=false
* fix: do not try to inflate a fresh lockfile
* fix(diff): walk target children if root is a link
* chore: @npmcli/package-json refactor
|
|
this didn't get ran when we added the new package-json package
|
|
This is a papercut that has been driving me crazy when debugging
ERESOLVE issues. It's not particularly useful to just say something is
"invalid", without showing which module's dependency is not being met.
With this commit, it prints all the packages that depend on that
dependency and do not have their required version met.
This does print multiple times for deduped deps that are invalid, but
if we skip the printing of the `invalid` label for deduped deps, we end
up losing information that is only detected later in the tree walk.
PR-URL: https://github.com/npm/cli/pull/3460
Credit: @isaacs
Close: #3460
Reviewed-by: @nlf
|
|
Refactor set-script and init to use @npmcli/package-json as a uniformed
way to update and save package.json files.
Fixes: https://github.com/npm/cli/issues/3234
Relates to: https://github.com/npm/statusboard/issues/368
PR-URL: https://github.com/npm/cli/pull/3455
Credit: @ruyadorno
Close: #3455
Reviewed-by: @nlf
|
|
We had tied the usage of commands to snapshots, but not the full config
descriptions. This will prevent us from landing changes to the config
descriptions that don't also result in rebuilt `.md` help docs.
Closes https://github.com/npm/statusboard/issues/374
PR-URL: https://github.com/npm/cli/pull/3459
Credit: @wraithgar
Close: #3459
Reviewed-by: @nlf
|
|
|
|
Consolidates existing "real npm" mocks, labels it real. We can now
migrate tests to this one at a time instead of trying to make one giant
PR that does it all at once.
PR-URL: https://github.com/npm/cli/pull/3458
Credit: @wraithgar
Close: #3458
Reviewed-by: @nlf
|
|
Tests for cli now use the real npm
PR-URL: https://github.com/npm/cli/pull/3451
Credit: @wraithgar
Close: #3451
Reviewed-by: @nlf
|
|
Cleaned the code up too
PR-URL: https://github.com/npm/cli/pull/3416
Credit: @wraithgar
Close: #3416
Reviewed-by: @nlf
|
|
This adds linting checking to our bin directory, fixes the linting
errors that generated, and cleans up lib/ls.js
to remove unused variables.
PR-URL: https://github.com/npm/cli/pull/3454
Credit: @Ivan12273
Close: #3454
Reviewed-by: @darcyclarke
|
|
PR-URL: https://github.com/npm/cli/pull/3450
Credit: @lumaxis
Close: #3450
Reviewed-by: @wraithgar
|
|
|
|
|
|
|
|
updates copy for package-lock-only and ls
PR-URL: https://github.com/npm/cli/pull/3435
Credit: @wraithgar
Close: #3435
Reviewed-by: @lukekarrys
|
|
|
|
|
|
|
|
|
|
This will allow users to specify a folder in which to save their
tarballs.
PR-URL: https://github.com/npm/cli/pull/3420
Credit: @wraithgar
Close: #3420
Reviewed-by: @ruyadorno
|
|
* use new npxCache option
|
|
This adds a new `npxCache` flatOption for libnpmexec to look for to put
its `_npx` content. libnpmexec will have to be patched to use that
value, and continue to pass `flatOptions.cache` to pacote et al.
The `flatOptions.cache` is the one that is intended to be passed down
into other modules, as it has the `_cacache` suffix attached. What was
happening before was that `npx` was creating a new alternate cache one
directory up from where everything else was, and also putting the `_npx`
content there. It is possible this is the source of at least some of
our "npx doesn't find the right versions" bugs.
PR-URL: https://github.com/npm/cli/pull/3430
Credit: @wraithgar
Close: #3430
Reviewed-by: @ruyadorno
|
|
`npm ls --prod` is currently not omitting devDependencies for configured
workspaces, this changes it by properly checking for the tweaked
`currentDepth` value instead of root check that was in place.
Fixes: https://github.com/npm/cli/issues/3382
PR-URL: https://github.com/npm/cli/pull/3429
Credit: @ruyadorno
Close: #3429
Reviewed-by: @wraithgar
|
|
* fix: implement cache modes correctly
|
|
The behavior for installing global versions if found has never been part
of npm@7.
PR-URL: https://github.com/npm/cli/pull/3418
Credit: @wraithgar
Close: #3418
Reviewed-by: @lukekarrys
|
|
* fix(inventory) handle old and british forms of 'license'
* fix: removes [_complete] check to apply correct metadata
* ensure node.fsParent is not set to node itself
* fix extraneous deps on load-actual
|
|
npm-package-arg is doing things more properly now so the tests should
reflect that
PR-URL: https://github.com/npm/cli/pull/3426
Credit: @wraithgar
Close: #3426
Reviewed-by: @ruyadorno
|
|
* fix: Make file: URLs (mostly) RFC 8909 compliant
|
|
This enables using the virtual tree instead of node_modules.
PR-URL: https://github.com/npm/cli/pull/3408
Credit: @G-Rath
Close: #3408
Reviewed-by: @isaacs
|
|
* fix(retrieve-tag): pass match in a way git accepts
|
|
PR-URL: https://github.com/npm/cli/pull/3423
Credit: @gurdiga
Close: #3423
Reviewed-by: @ruyadorno
|
|
Also includes a preliminary refactor to consolidate workspace logic now
that every command that supports workspaces has it implemented.
PR-URL: https://github.com/npm/cli/pull/3417
Credit: @wraithgar
Close: #3417
Reviewed-by: @ruyadorno
|
|
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
|
|
- Setup ./packages/* as workspaces
- Moved source from: https://github.com/npm/libnpmdiff to
./packages/libnpmdiff
- Added CI target for workspaces
Relates to: https://github.com/npm/statusboard/issues/362
PR-URL: https://github.com/npm/cli/pull/3386
Credit: @ruyadorno
Close: #3386
Reviewed-by: @wraithgar
|
|
`npm link <pkg>` is meant to be used as a way to link a local package to
an install tree and it's very surprising to users that it may prune
extraneous deps from the project.
This change switches the default behavior to avoid pruning deps when
reifying the dependencies in npm link.
Fixes: https://github.com/npm/cli/issues/2554
PR-URL: https://github.com/npm/cli/pull/3399
Credit: @ruyadorno
Close: #3399
Reviewed-by: @ljharb, @nlf
|
|
|
|
|
|
|