diff options
| author | Michał Gołębiowski-Owczarek <m.goleb@gmail.com> | 2019-10-05 18:48:27 +0200 |
|---|---|---|
| committer | Michał Gołębiowski-Owczarek <m.goleb@gmail.com> | 2019-10-05 18:50:18 +0200 |
| commit | c7a5e1bd797809f2aab1214d5f90ec94da9fdbd6 (patch) | |
| tree | 91fdc1bd25c01452b151ba8137eda80f46c6d523 | |
| parent | 279d2e979ebb26d846036e0b02fd6070a013ed3b (diff) | |
| download | jquery-c7a5e1bd797809f2aab1214d5f90ec94da9fdbd6.tar.gz jquery-c7a5e1bd797809f2aab1214d5f90ec94da9fdbd6.zip | |
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)
| -rw-r--r-- | build/release/dist.js | 3 |
1 files changed, 2 insertions, 1 deletions
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 |