bootstrap: Use the --no-save flag in CI

Prevents the cache from becoming outdated
when e.g. The "version" field of `package.json`
is updated and checked in, but not updated or
checked in for `package-lock.json`.

Author: DeeDeeG

script/lib/install-apm.js

@@ -9,7 +9,7 @@ module.exports = function(ci) {
   // npm ci leaves apm with a bunch of unmet dependencies
   childProcess.execFileSync(
     CONFIG.getNpmBinPath(),
-    ['--global-style', '--loglevel=error', 'install'],
+    ['--global-style', '--loglevel=error', 'install', ci ? '--no-save' : ''],
     { env: process.env, cwd: CONFIG.apmRootPath }
   );
 };

script/lib/run-apm-install.js

@@ -11,7 +11,7 @@ module.exports = function(packagePath, ci, stdioOptions) {
   // Set our target (Electron) version so that node-pre-gyp can download the
   // proper binaries.
   installEnv.npm_config_target = CONFIG.appMetadata.electronVersion;
-  childProcess.execFileSync(CONFIG.getApmBinPath(), ['install'], {
+  childProcess.execFileSync(CONFIG.getApmBinPath(), ['install', ci ? '--no-save' : ''], {
     env: installEnv,
     cwd: packagePath,
     stdio: stdioOptions || 'inherit'