Merge pull request #148 from opentable/multiple-publish-bug

only remove the package once published to all registries
opencomponents · Nov 24, 2015 · 37ae3f0 · 37ae3f0
2 parents 69d0d1b + 87b345d
commit 37ae3f0
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/cli/facade/publish.js b/cli/facade/publish.js
@@ -102,7 +102,7 @@ module.exports = function(dependencies){
           }
         } else {
           logger.log(format(strings.messages.cli.PUBLISHED, options.route.green).yellow);
-          return local.cleanup(options.path, callback);
+          return callback();
         }
       });
     };
@@ -124,7 +124,9 @@ module.exports = function(dependencies){
               componentRoute = format('{0}/{1}/{2}', registryNormalised, component.name, component.version);
 
           putComponentToRegistry({ route: componentRoute, path: compressedPackagePath}, done);
-        }, reallyDoneThisTime);
+        }, function(err){
+          local.cleanup(compressedPackagePath, reallyDoneThisTime);
+        });
       });
     });
   };