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 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.
|
|
|
|
PR-URL: https://github.com/npm/cli/pull/1520
Credit: @claudiahdz
Close: #1520
Reviewed-by: @ruyadorno
|
|
|
|
PR-URL: https://github.com/npm/cli/pull/1497
Credit: @claudiahdz
Close: #1497
Reviewed-by: @isaacs
|
|
PR-URL: https://github.com/npm/cli/pull/1488
Credit: @claudiahdz
Close: #1488
Reviewed-by: @ruyadorno
|