From: Michał Gołębiowski-Owczarek <m.goleb@gmail.com>
Date: Sat, 5 Oct 2019 16:48:27 +0000 (+0200)
Subject: Build: Support jquery-release --dry-run flag
X-Git-Tag: 3.5.0~37
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=c7a5e1bd797809f2aab1214d5f90ec94da9fdbd6;p=jquery.git

Build: Support jquery-release --dry-run flag

Without this change passing `--dry-run` to jquery-release still pushes to the
jquery-dist repository which is dangerous as one can assume `--dry-run` to be
safe from external side effects.

Close gh-4498

(cherry picked from commit d7d0b52bda74486f2351baa9d03ca4534de0d733)
---

diff --git a/build/release/dist.js b/build/release/dist.js
index ca7922628..ef4ee7c92 100644
--- a/build/release/dist.js
+++ b/build/release/dist.js
@@ -135,7 +135,8 @@ module.exports = function( Release, files, complete ) {
 		Release.chdir( Release.dir.dist );
 
 		console.log( "Pushing release to dist repo..." );
-		Release.exec( "git push " + distRemote + " master --tags",
+		Release.exec( "git push " + ( Release.isTest ? " --dry-run " : "" ) +
+			distRemote + " master --tags",
 			"Error pushing master and tags to git repo." );
 
 		// Set repo for npm publish