From 14c19cf1a8dc3845f3f6b82f27be75334b2591e3 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 e2008f855..968defd7b 100644 --- a/build/release.js +++ b/build/release.js @@ -301,13 +301,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