chore(release): Release zip is now named according to release

[Finishes #83207572]

Signed-off-by: Paul Meskers <pmeskers@pivotal.io>

.gitignore

@@ -16,7 +16,7 @@ node_modules/
 /vendor/
 .idea/
 /dist/
-/dist.zip
+*.zip
 
 ## Specific to RubyMotion:
 .dat*

tasks/release.js

@@ -36,10 +36,18 @@ gulp.task('release', [
 
 gulp.task('_zip', [
   'assets',
-], function(){
-  return gulp.src('dist/**/*')
-    .pipe(zip('dist.zip'))
-    .pipe(gulp.dest('./'));
+  '_addVersionRelease',
+], function(done){
+  releaseHelper.getNewReleaseName
+  .then(function(newReleaseName) {
+    gulp.src(['release/' + newReleaseName + '/**/*'])
+      .pipe(zip(newReleaseName + '.zip'))
+      .pipe(gulp.dest('./'))
+      .on('end', done);
+  })
+  .fail(function(err) {
+    errorHandler.handleError(err, {callback: done});
+  });
 });
 
 gulp.task('_changelog', function(done) {
@@ -73,10 +81,10 @@ gulp.task('_bumpPackage', ['assets'], function(done) {
 });
 
 gulp.task('_addVersionRelease', ['assets'], function(done) {
-  releaseHelper.getNewVersion
-  .then(function(newVersion) {
+  releaseHelper.getNewReleaseName
+  .then(function(newReleaseName) {
     gulp.src('dist/**/*')
-      .pipe(gulp.dest('release/pui-v' + newVersion + '/'))
+      .pipe(gulp.dest('release/' + newReleaseName + '/'))
       .on('end', done);
   })
   .fail(function(err) {

tasks/releaseHelper.js

@@ -33,6 +33,21 @@ var getNewTagName = function() {
   return deferred.promise;
 }();
 
+var getNewReleaseName = function() {
+  var deferred = q.defer();
+
+  getNewVersion
+  .then(function(newVersion) {
+    deferred.resolve('pui-v' + newVersion);
+  })
+  .fail(function(err) {
+    deferred.reject(err);
+  });
+
+  return deferred.promise;
+}();
+
+
 var getVersionChanges = function() {
   var deferred = q.defer();
 
@@ -59,6 +74,7 @@ var getVersionChanges = function() {
 module.exports = {
   getNewVersion: getNewVersion,
   getNewTagName: getNewTagName,
+  getNewReleaseName: getNewReleaseName,
   getVersionChanges: getVersionChanges
 };