diff options
author | Dave <dave@jut.io> | 2016-01-22 07:41:04 +0300 |
---|---|---|
committer | Rebecca Turner <me@re-becca.org> | 2016-01-29 01:58:46 +0300 |
commit | 9ab8b8d047792612ae7f9a6079745d51d5283a53 (patch) | |
tree | c4db7f78b2370356ad5b11fd6605fc80276ed0c0 /lib | |
parent | cb9df5a37091e06071d8704b629e7ebaa41c37fe (diff) |
ls: don't subtract dev from production deps
If a package is in dependencies and devDependencies,
filterByEnv removes it from ls --production. This doesn't
make sense, since the package is still a prod dependency.
Fixes: #10820
PR-URL: https://github.com/npm/npm/pull/11245
Credit: @davidvgalbraith
Reviewed-By: @iarna
Diffstat (limited to 'lib')
-rw-r--r-- | lib/ls.js | 1 |
1 files changed, 0 insertions, 1 deletions
@@ -120,7 +120,6 @@ function filterByEnv (data) { var devDependencies = data.devDependencies || [] Object.keys(data.dependencies).forEach(function (name) { var keys = Object.keys(devDependencies) - if (production && !dev && keys.indexOf(name) !== -1) return if (dev && !production && keys.indexOf(name) === -1) return if (!dev && keys.indexOf(name) !== -1 && data.dependencies[name].missing) return dependencies[name] = data.dependencies[name] |