Age | Commit message (Collapse) | Author |
|
Instead of files randomly requiring the npm singleton,
we pass it where it needs to go so that tests don't need
to do so much require mocking everywhere
PR-URL: https://github.com/npm/cli/pull/2772
Credit: @wraithgar
Close: #2772
Reviewed-by: @ruyadorno
|
|
We also removed the "none" script because we handle a missing
script just fine. There is no need to put an empty one in
PR-URL: https://github.com/npm/cli/pull/2759
Credit: @wraithgar
Close: #2759
Reviewed-by: @nlf
|
|
If an env script is already defined, run that
instead of the default.
PR-URL: https://github.com/npm/cli/pull/2655
Credit: @isaacs
Close: #2655
Reviewed-by: @ljharb
|
|
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.
|
|
|
|
Fixes issue #2023
PR-URL: https://github.com/npm/cli/pull/2086
Credit: @macno
Close: #2086
Reviewed-by: @nlf
|
|
PR-URL: https://github.com/npm/cli/pull/1987
Credit: @G-Rath
Close: #1987
Reviewed-by: @ruyadorno
|
|
Implement support to `ignoreScripts` config option in `npm run-script`
allowing for skipping pre and post hooks when used as discussed in
RFC-0029.
ref: https://github.com/npm/rfcs/blob/ed67c2bc41873d8f1954f8ce4a8f00437f5f14eb/accepted/0029-add-ability-to-skip-hooks.md
PR-URL: https://github.com/npm/cli/pull/1718
Credit: @ruyadorno
Close: #1718
Reviewed-by: @isaacs
|
|
|