Age | Commit message (Collapse) | Author |
|
PR-URL: https://github.com/npm/cli/pull/2361
Credit: @Edu93Jer
Close: #2361
Reviewed-by: @isaacs
EDIT(@isaacs): amended to match some of the fixes to our current style
conventions
|
|
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.
|
|
|
|
Pending unit tests still, but at least it works to a rough approximation
now, instead of not working at all.
|
|
PR-URL: https://github.com/npm/cli/pull/1543
Credit: @isaacs
Close: #1543
Reviewed-by: @ruyadorno
|
|
Way too opinionated and hard to configure. Copied out standard's config
as a starting point, and tweaked a few things that don't make sense for
the npm cli project.
|