From b06d39726f092682c3f7630bf774b798c5c531a0 Mon Sep 17 00:00:00 2001 From: Jonas Hermsmeier Date: Fri, 21 Apr 2017 18:33:27 +0200 Subject: [PATCH] chore(scripts): Remove prune from shrinkwrap cleanup (#1338) Newer npm versions (4.4+ I believe) even remove dependencies not shrinkwrapped yet while pruning causing newly installed dependencies to not always be shrinkwrapped / updated in the shrinkwrap. Removing the prune allows for this to work properly again and in the future, with the drawback that care must be taken to not have extraneous dependencies in the module tree. Change-Type: patch --- scripts/clean-shrinkwrap.js | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/scripts/clean-shrinkwrap.js b/scripts/clean-shrinkwrap.js index bbaccf87..6fbca398 100644 --- a/scripts/clean-shrinkwrap.js +++ b/scripts/clean-shrinkwrap.js @@ -38,7 +38,5 @@ const npm = (command) => { npm([ 'rm', '--ignore-scripts' ].concat(shrinkwrapIgnore)) .once('close', () => { - npm([ 'prune' ]).once('close', () => { - console.log('Done.'); - }); + console.log('Done.'); });