Age | Commit message (Collapse) | Author |
|
PR-URL: https://github.com/npm/npm/pull/9929
|
|
PR-URL: https://github.com/npm/npm/pull/9929
|
|
PR-URL: https://github.com/npm/npm/pull/9929
|
|
PR-URL: https://github.com/npm/npm/pull/9929
|
|
|
|
When running `install --save` in a folder with a shrinkwrap that has
dev-only dependencies in it, it's important that we don't then throw
away the dev dependencies just because they didn't say
`install --save --also=dev`.
PR-URL: https://github.com/npm/npm/pull/9992
Fixes: #9647
|
|
package maintainers
PR-URL: https://github.com/npm/npm/pull/9950
|
|
PR-URL: https://github.com/npm/npm/pull/9954
|
|
PR-URL: https://github.com/npm/npm/pull/9927
|
|
PR-URL: https://github.com/npm/npm/pull/9928
|
|
This reverts commit 22a3af055f934f8de20ebd5f4a286447dda84f81.
|
|
If an optional dependency was not installed, shrinkwrap fails with a
message saying that a required dependency is missing. This patch checks
whether a missing dependency was listed in the optionalDependencies.
PR-URL: https://github.com/npm/npm/pull/9879
|
|
the NODE_ENV.
The following is the condition of setting the value of the development / production option.
npm install
+ dev: true
+ prod: true
--only=dev
+ dev: true
+ prod: false
--only=prod || NODE_ENV=production
+ dev: false
+ prod: true
PR-URL: https://github.com/npm/npm/pull/9835
Fixes: #9463
|
|
Fixes #9669
PR-URL: https://github.com/npm/npm/pull/9715
|
|
PR-URL: https://github.com/npm/npm/pull/9553
|
|
PR-URL: //github.com/npm/npm/pull/9890
Fixes: #9766
|
|
PR-URL: //github.com/npm/npm/pull/9890
Fixes: #9766
|
|
PR-URL: //github.com/npm/npm/pull/9890
Fixes: #9766
|
|
Bring consistent guarding against null/undefined and consistent
business logic.
PR-URL: //github.com/npm/npm/pull/9890
Fixes: #9766
|
|
Previously this was being done implicitly via `build.linkStuff`– we're now
avoiding that.
PR-URL: https://github.com/npm/npm/pull/9891
Fixes: #9643
Fixes: #9664
|
|
PR-URL: https://github.com/npm/npm/pull/9892
|
|
Lodash's deep-clone is being updated to not have the catostrophic scalaing issues
it had when this patch was written, but even still, using something tailor-made
will necessarily be faster.
PR-URL: https://github.com/npm/npm/pull/9803
Fixes: #8826
|
|
The clone we were doing to save ourselves from mutation proved to be
excessively slow. While lodash is being updated to not have the
same scaling issues, it was determined that we weren't gaining
anything from not mutating here, so there was no reason to pay
even a lesser price.
PR-URL: https://github.com/npm/npm/pull/9803
Fixes: #8826
|
|
PR-URL: https://github.com/npm/npm/pull/9471
|
|
PR-URL: https://github.com/npm/npm/pull/9667
|
|
PR-URL: https://github.com/npm/npm/pull/9667
|
|
PR-URL: https://github.com/npm/npm/pull/9741
|
|
PR-URL: https://github.com/npm/npm/pull/9744
Fixes: #9695
|
|
|
|
|
|
Fixes: #9544
PR-URL: https://github.com/npm/npm/pull/9549
|
|
(provided they are newer than 0.7.9)
PR-URL: https://github.com/npm/npm/pull/9506
Fixes: #9505
|
|
PR-URL: https://github.com/npm/npm/pull/9537
Fixes: 9394
|
|
Reports of `EISDIR` on the issue tracker are almost exclusively caused by users
trying to install something that does not have a `package.json`. However, it is
clearly difficult for many users to discern this simple problem from the error
code alone. So let's expand upon it.
PR-URL: https://github.com/npm/npm/pull/9396
|
|
Followup-To: https://github.com/npm/npm/issues/1648
Fixes: https://github.com/npm/npm/issues/8517
PR-URL: https://github.com/npm/npm/pull/9409
PR-URL: https://github.com/npm/npm/pull/8841
|
|
PR-URL: https://github.com/npm/npm/pull/9409
|
|
PR-URL: https://github.com/npm/npm/pull/9369
Fixes: #9308
|
|
PR-URL: https://github.com/npm/npm/pull/9367
Fixes: #9205
|
|
To make debugging of lifecycle scripts easier
PR-URL: https://github.com/npm/npm/pull/9227
|
|
The install order is determined by:
1) The location of the thing to be installed relative to the root module.
2) Dependency order, such that a -> b -> c will install modules as: c, b, a
1 is deterministic, regardless of what's being installed.
But 2 can change the order of things higher in the dep tree. Eg, b, or a
might get sorted earlier if c requires them. This mostly doesn't matter,
but it does mean that if you have two modules with conflicting bins, they
_can_ get installed in different orders. This changes sorts all of the top
level modules to be LAST, in location order (1), and then sorts all the rest
per (2). This ensures that top level modules have a deterministic install
order. (Non top level modules can't have bin conflicts as that's treated
the same as a version conflict and the conflicting module would be hoisted.)
PR-URL: https://github.com/npm/npm/pull/9274
Fixes: #8995
|
|
logical tree
PR-URL: https://github.com/npm/npm/pull/9344
Fixes: #9113
|
|
Previously I was using the path of the module to be updated. But no, silly,
we want the path that CONTAINS it.
PR-URL: https://github.com/npm/npm/pull/9303
Fixes: #9095
|
|
PR-URL: https://github.com/npm/npm/pull/9343
Fixes: #9113
|
|
We have special code to disable the progress bar on travis (and other CI) to
save ourselves and our users a lot of unhelpful output. Unfortunately as
this test didn't take that into account, it immediately exploded in this
envs.
So we fix that up, PLUS we add tests for the various env options for
suppressing progress bars.
PR-URL: https://github.com/npm/npm/pull/9304
|
|
/d disables cmd.exe AutoRuns feature, where registry entry points to the
script that will be executed when cmd.exe starts. Surprisingly a lot of
users hase "CD \" or similar command there which causes lifecycle
scripts to fail, since relative path to the script no longer works (and
package.json has no way to deduce absolute path)
/s enables handling of quotes, so that you can have "C:\Program
Files\Node\node.exe" quoted in your script. Node's child_process.exec()
is doing the same. We prevent additional quoting by libuv already by
setting uv_spawn() flag UV_PROCESS_WINDOWS_VERBATIM_ARGUMENTS.
If those flags are not set there is no way package.json can reliably
tell us to run "node" executable with a relative script path:
"scripts": {
"install": "node scripts/install.js"
}
Without /d "node" invocation may end up in a random directory as a
result of the AutoRuns command.
Fixes: #8751
Fixes: #7333
PR-URL: https://github.com/npm/npm/pull/9245
|
|
npm@1.x didn't include configuration values prefixed with an underscore
in the environment passed to chiled processes. npm@2.x includes the
notion of scoped configuration, where a nerfed URL can be used to prefix
configuration, some of which might be underscore-prefixed without the
scope. Add that behavior to npm@2 and close this regression.
PR-URL: https://github.com/npm/npm/pull/9348
|
|
PR-URL: https://github.com/npm/npm/pull/9011
|
|
PR-URL: https://github.com/npm/npm/pull/9011
|
|
PR-URL: https://github.com/npm/npm/pull/9011
|
|
PR-URL: https://github.com/npm/npm/pull/9198
|