From 1a0361a57d076fd0cee0348a3fc7a5bea1b026fd Mon Sep 17 00:00:00 2001 From: Maxim Belkin Date: Wed, 20 Nov 2019 12:38:50 -0600 Subject: [PATCH] Don't check dry run status where it is not needed --- Library/Homebrew/dev-cmd/bump-formula-pr.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Library/Homebrew/dev-cmd/bump-formula-pr.rb b/Library/Homebrew/dev-cmd/bump-formula-pr.rb index ed644e28d0..58cb293265 100644 --- a/Library/Homebrew/dev-cmd/bump-formula-pr.rb +++ b/Library/Homebrew/dev-cmd/bump-formula-pr.rb @@ -383,7 +383,7 @@ module Homebrew def forked_repo_info(tap_full_name) response = GitHub.create_fork(tap_full_name) rescue GitHub::AuthenticationFailedError, *GitHub.api_errors => e - formula.path.atomic_write(backup_file) unless args.dry_run? + formula.path.atomic_write(backup_file) odie "Unable to fork: #{e.message}!" else # GitHub API responds immediately but fork takes a few seconds to be ready.