fix(ci): pay attention to --ignore-scripts

Added a test for install too for this specific condition

PR-URL: https://github.com/npm/cli/pull/2455
Credit: @wraithgar
Close: #2455
Reviewed-by: @isaacs

1 files changed, 21 insertions, 19 deletions

diff --git a/lib/ci.js b/lib/ci.js
index 2917899c8..89c6b8f42 100644
--- a/lib/ci.js
+++ b/lib/ci.js
@@ -21,7 +21,7 @@ const ci = async () => {
   }
 
   const where = npm.prefix
-  const { scriptShell } = npm.flatOptions
+  const { scriptShell, ignoreScripts } = npm.flatOptions
   const arb = new Arborist({ ...npm.flatOptions, path: where })
 
   await Promise.all([
@@ -39,24 +39,26 @@ const ci = async () => {
   await arb.reify({ ...npm.flatOptions, save: false })
 
   // run the same set of scripts that `npm install` runs.
-  const scripts = [
-    'preinstall',
-    'install',
-    'postinstall',
-    'prepublish', // XXX should we remove this finally??
-    'preprepare',
-    'prepare',
-    'postprepare',
-  ]
-  for (const event of scripts) {
-    await runScript({
-      path: where,
-      args: [],
-      scriptShell,
-      stdio: 'inherit',
-      stdioString: true,
-      event,
-    })
+  if (!ignoreScripts) {
+    const scripts = [
+      'preinstall',
+      'install',
+      'postinstall',
+      'prepublish', // XXX should we remove this finally??
+      'preprepare',
+      'prepare',
+      'postprepare',
+    ]
+    for (const event of scripts) {
+      await runScript({
+        path: where,
+        args: [],
+        scriptShell,
+        stdio: 'inherit',
+        stdioString: true,
+        event,
+      })
+    }
   }
   await reifyFinish(arb)
 }