CDN: Confirm before push

Fixes gh-24

Author: jzaefferer

lib/cdn.js

@@ -1,14 +1,34 @@
-var shell = require( "shelljs" );
+var shell = require( "shelljs" ),
+	fs = require( "fs" );
 
 module.exports = function( Release ) {
 	var realRemote = "git@github.com:jquery/codeorigin.jquery.com.git",
 		testRemote = "git@github.com:jquery/fake-cdn.git";
 
+	function projectCdn() {
+		var jqueryCdn = Release._cloneCdnRepo() + "/cdn";
+		if ( Release.project === "jquery" ) {
+			return jqueryCdn;
+		}
+		if ( Release.project === "qunit" ) {
+			return jqueryCdn + "/qunit";
+		}
+		if ( /^jquery-/.test( Release.project ) ) {
+			return jqueryCdn + "/" + Release.project.substring( 7 ) +
+				"/" + Release.newVersion;
+		}
+		return jqueryCdn + "/" + Release.project + "/" + Release.newVersion;
+	}
+
 	Release.define({
 		_cloneCdnRepo: function() {
 			var local = Release.dir.base + "/codeorigin.jquery.com",
 				remote = Release.isTest ? testRemote : realRemote;
 
+			if ( fs.existsSync( local ) ) {
+				return local;
+			}
+
 			console.log( "Cloning " + remote.cyan + "..." );
 			Release.chdir( Release.dir.base );
 			Release.git( "clone " + remote + " " + local, "Error cloning CDN repo." );
@@ -17,31 +37,24 @@ module.exports = function( Release ) {
 			return local;
 		},
 
-		_pushToCdn: function() {
-			var projectCdn,
-				jqueryCdn = Release._cloneCdnRepo() + "/cdn",
+		_copyCdnArtifacts: function() {
+			var targetCdn = projectCdn(),
 				releaseCdn = Release.dir.repo + "/dist/cdn",
 				commitMessage = Release.project + ": Added version " + Release.newVersion;
 
-			if ( Release.project === "jquery" ) {
-				projectCdn = jqueryCdn;
-			} else if ( Release.project === "qunit" ) {
-				projectCdn = jqueryCdn + "/qunit";
-			} else if ( /^jquery-/.test( Release.project ) ) {
-				projectCdn = jqueryCdn + "/" + Release.project.substring( 7 ) +
-					"/" + Release.newVersion;
-			} else {
-				projectCdn = jqueryCdn + "/" + Release.project + "/" + Release.newVersion;
-			}
-
 			Release.chdir( Release.dir.base );
-			shell.mkdir( "-p", projectCdn );
-			shell.cp( "-r", releaseCdn + "/*", projectCdn );
+			console.log( "Copying files from " + releaseCdn.cyan + " to " + targetCdn.cyan + "." );
+			shell.mkdir( "-p", targetCdn );
+			shell.cp( "-r", releaseCdn + "/*", targetCdn );
 
 			console.log( "Adding files..." );
-			Release.chdir( projectCdn );
+			Release.chdir( targetCdn );
 			Release.git( "add ." , "Error adding files." );
 			Release.git( "commit -m '" + commitMessage + "'" , "Error commiting files." );
+		},
+
+		_pushToCdn: function() {
+			Release.chdir( projectCdn() );
 			Release.git( "push", "Error pushing to CDN." );
 			console.log();
 		}

release.js

@@ -41,6 +41,8 @@ commonTasks = [
 	Release._pushRelease,
 
 	Release._section( "publishing to jQuery CDN" ),
+	Release._copyCdnArtifacts,
+	Release.confirmReview,
 	Release._pushToCdn
 ];