From c7a5e1bd797809f2aab1214d5f90ec94da9fdbd6 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Micha=C5=82=20Go=C5=82=C4=99biowski-Owczarek?= Date: Sat, 5 Oct 2019 18:48:27 +0200 Subject: [PATCH] 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) --- build/release/dist.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) 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 -- 2.39.5