fix: include all dep types when building for prepare

Fixes: npm/cli#2920
npm · Jun 18, 2021 · b76f51a · b76f51a
1 parent 745265a
commit b76f51a
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 0 deletions.
diff --git a/lib/fetcher.js b/lib/fetcher.js
@@ -119,6 +119,10 @@ class FetcherBase {
       '--no-progress',
       '--no-save',
       '--no-audit',
+      // override any omit settings from the environment
+      '--include=dev',
+      '--include=peer',
+      '--include=optional',
     ]
   }
 

diff --git a/tap-snapshots/test/fetcher.js-fake-sudo.test.cjs b/tap-snapshots/test/fetcher.js-fake-sudo.test.cjs
@@ -43,6 +43,9 @@ Array [
   "--no-progress",
   "--no-save",
   "--no-audit",
+  "--include=dev",
+  "--include=peer",
+  "--include=optional",
 ]
 `
 
@@ -56,6 +59,9 @@ Array [
   "--no-progress",
   "--no-save",
   "--no-audit",
+  "--include=dev",
+  "--include=peer",
+  "--include=optional",
 ]
 `
 

diff --git a/tap-snapshots/test/fetcher.js.test.cjs b/tap-snapshots/test/fetcher.js.test.cjs
@@ -43,6 +43,9 @@ Array [
   "--no-progress",
   "--no-save",
   "--no-audit",
+  "--include=dev",
+  "--include=peer",
+  "--include=optional",
 ]
 `
 
@@ -56,6 +59,9 @@ Array [
   "--no-progress",
   "--no-save",
   "--no-audit",
+  "--include=dev",
+  "--include=peer",
+  "--include=optional",
 ]
 `