Age | Commit message (Collapse) | Author |
|
This is a huge set of mostly mechanical changes. Going forward, all
changes to the npm source base are expected to comply with `standard`,
and it's been integrated into the test suite to enforce that. There are
a few notes below about specific classes of changes that need to be
handled specially for npm's code base.
standard: "Expected error to be handled."
`standard` only expects errors spelled "err" to be handled.
`npm-registry-mock` never actually invokes its callback with an error,
so in some cases I just changed it to be spelled "er" and called it
good.
standard: "Expected a "break" statement before 'case'."
This behavior is actually on purpose, and I don't feel like rewriting
the affected code right now (or, you know, ever). So I added code
comments disabling the checks in the three applicable changes.
standard: "x is a function."
Rebinding functions created via declarations (as opposed to expressions)
is a no-no?
PR-URL: https://github.com/npm/npm/pull/8668
|
|
When idealizing the tree for dedupe, only clone and load-extraneous
|
|
|
|
Moving wasn't correctly unbuilding which was resulting in orphaned
.bin directories, which in turn resulted in turn resulted in broken
node_module's folders.
Deduped removes weren't removing all of the children of the thing
being removed.
Moving wasn't explicitly moving all of the children of the thing
being moved.
|
|
|
|
|
|
This is safe to do globally as deps of globals are NEVER hoisted out of
the global. Each global is self contained.
|
|
put @ inside <@scope>
simplify completion usage
add [@<version>] to edit
remove extraneous from install
|
|
|
|
Test is now self-contained as well.
|
|
|
|
Makes a *lot* clearer when the CLI is doing something with auth
implications, by explicitly pass auth to registry client. Moves lookup
of credentials into mapRegistryToURI, which makes the code more
consistent.
|
|
|
|
|
|
|
|
|
|
|
|
Fixes issue #4675
|
|
Re #4691
|
|
`npm dedupe` fails silently on windows due to incorrect
path splitting and joining without these changes.
|
|
|
|
|
|
|
|
Re: Colingo/date-now#1
|
|
This is another check against the kind of error that led to #3065
|
|
|
|
|
|
|
|
devDeps are ignored. Extraneous deps are treated as if they are
required with '' as the range.
|
|
|