When in flat mode don't add unused dependencies - fixes #2781 (#3251)

yarnpkg · Apr 25, 2017 · 2458758 · 2458758
1 parent 0150890
commit 2458758
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/cli/commands/install.js b/src/cli/commands/install.js
@@ -239,6 +239,12 @@ export class Install {
         if (ignoreUnusedPatterns && !isUsed) {
           return;
         }
+        // We only take unused dependencies into consideration to get deterministic hoisting.
+        // Since flat mode doesn't care about hoisting and everything is top level and specified then we can safely
+        // leave these out.
+        if (this.flags.flat && !isUsed) {
+          return;
+        }
         const depMap = json[depType];
         for (const name in depMap) {
           if (excludeNames.indexOf(name) >= 0) {