From 21a1e7d38f7be3356d9eee9da4a468dcf8c6a8f9 Mon Sep 17 00:00:00 2001 From: Timmy Willison Date: Fri, 15 Nov 2013 10:48:49 -0500 Subject: [PATCH] Release script: Add dist files in multiple commits for a clean final commit with a small diff --- build/release.js | 18 +++++++++++++++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff --git a/build/release.js b/build/release.js index b7c942fac..9dfeb8d5b 100644 --- a/build/release.js +++ b/build/release.js @@ -300,13 +300,25 @@ function checkoutCommit( next ) { function commitDistFiles( next ) { // Remove scripts property from package.json + // Building and bower are irrelevant as those files will be committed + // Makes for a clean npm install var pkgClone = readJSON( "package.json" ); delete pkgClone.scripts; fs.writeFileSync( "package.json", JSON.stringify( pkgClone, null, "\t" ) ); // Add files to be committed - // Use force to add normally ignored files - git( [ "add", "-f", "package.json", devFile, minFile, mapFile, sizzleLoc ], function() { - git( [ "commit", "-m", releaseVersion ], next, debug ); + git( [ "add", "package.json" ], function() { + git( [ "commit", "-m", "Remove scripts property from package.json" ], function() { + // Add sizzle in a separate commit to avoid a big diff + // Use force to add normally ignored files + git( [ "add", "-f", sizzleLoc ], function() { + git( [ "commit", "-m", "Add sizzle" ], function() { + // Add jquery files for distribution in a final commit + git( [ "add", "-f", devFile, minFile, mapFile ], function() { + git( [ "commit", "-m", releaseVersion ], next, debug ); + }, debug ); + }, debug ); + }, debug ); + }, debug ); }, debug ); } -- 2.39.5