From 2834cca9fafb4a2bb51993786f5f4f1fe2631bd3 Mon Sep 17 00:00:00 2001 From: Feross Aboukhadijeh Date: Fri, 6 Sep 2019 00:11:52 -0700 Subject: [PATCH] package: use `npm ci`, remove `npm dedupe` `npm ci` does not modify the package-lock.json file, which is exactly what we want during packaging time. Otherwise, on Windows, npm will attempt to remove the mac-only dependencies like appdmg from package-lock.json. --- bin/package.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bin/package.js b/bin/package.js index 6b0566df..6715bd9a 100755 --- a/bin/package.js +++ b/bin/package.js @@ -39,8 +39,8 @@ const argv = minimist(process.argv.slice(2), { function build () { console.log('Reinstalling node_modules...') rimraf.sync(NODE_MODULES_PATH) - cp.execSync('npm install', { stdio: 'inherit' }) cp.execSync('npm dedupe', { stdio: 'inherit' }) + cp.execSync('npm ci', { stdio: 'inherit' }) console.log('Nuking dist/ and build/...') rimraf.sync(DIST_PATH)