diff options
author | isaacs <i@izs.me> | 2019-07-24 23:00:03 -0700 |
---|---|---|
committer | Michaƫl Zasso <targos@protonmail.com> | 2019-08-06 09:05:32 +0200 |
commit | d7d321b071789f08c65dbb11a0e4b3b6a299af44 (patch) | |
tree | 1015b8c5da3632fc986b56051a4853ac946c56f4 /deps/npm/lib/install/is-only-optional.js | |
parent | 37d27486fce50bd82b6b5095af880d435ed308f8 (diff) | |
download | node-new-d7d321b071789f08c65dbb11a0e4b3b6a299af44.tar.gz |
deps: upgrade npm to 6.10.2
PR-URL: https://github.com/nodejs/node/pull/28853
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Diffstat (limited to 'deps/npm/lib/install/is-only-optional.js')
-rw-r--r-- | deps/npm/lib/install/is-only-optional.js | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/deps/npm/lib/install/is-only-optional.js b/deps/npm/lib/install/is-only-optional.js index 81e227bae7..ea27eadcfa 100644 --- a/deps/npm/lib/install/is-only-optional.js +++ b/deps/npm/lib/install/is-only-optional.js @@ -11,11 +11,12 @@ function isOptional (node, seen) { if (seen.has(node) || node.requiredBy.length === 0) { return false } - seen = new Set(seen) seen.add(node) const swOptional = node.fromShrinkwrap && node.package._optional - return node.requiredBy.every(function (req) { + const result = node.requiredBy.every(function (req) { if (req.fakeChild && swOptional) return true return isOptDep(req, moduleName(node)) || isOptional(req, seen) }) + seen.delete(node) + return result } |