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 linter.
yolo
|
|
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.
|
|
`npm fund` human output was appending any items that had multiple
funding sources to the current package title as comma-separated names.
This commit fixes the problem by properly selecting the first item of a
each funding element and only using that as its index for printing the
human output tree representation.
PR-URL: https://github.com/npm/cli/pull/1717
Credit: @ruyadorno
Close: #1717
Reviewed-by: @isaacs
|
|
PR-URL: https://github.com/npm/cli/pull/1593
Credit: @ruyadorno
Close: #1593
Reviewed-by: @isaacs
EDIT(@isaacs): Added black background to the highlighted text so that it
won't disappear on light terminal settings.
|
|
Refactored `npm fund` tests to use new `test/lib/` unit tests structure.
ref: npm/statusboard#151
PR-URL: https://github.com/npm/cli/pull/1582
Credit: @ruyadorno
Close: #1582
Reviewed-by: @isaacs
|