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: 4.0.0-beta~245
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=d7d0b52bda74486f2351baa9d03ca4534de0d733;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
---

diff --git a/build/release/dist.js b/build/release/dist.js
index 577eb6404..0b1e04a7f 100644
--- a/build/release/dist.js
+++ b/build/release/dist.js
@@ -130,7 +130,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