Merge pull request #16508 from issyl0/handle-auth-errors
dev-cmd/bump: Don't fall over when retrieving PRs 404s
This commit is contained in:
commit
c5b35b87ab
@ -288,7 +288,13 @@ module Homebrew
|
|||||||
}
|
}
|
||||||
def retrieve_pull_requests(formula_or_cask, name, state:, version: nil)
|
def retrieve_pull_requests(formula_or_cask, name, state:, version: nil)
|
||||||
tap_remote_repo = formula_or_cask.tap&.remote_repo || formula_or_cask.tap&.full_name
|
tap_remote_repo = formula_or_cask.tap&.remote_repo || formula_or_cask.tap&.full_name
|
||||||
pull_requests = GitHub.fetch_pull_requests(name, tap_remote_repo, state: state, version: version)
|
pull_requests = begin
|
||||||
|
GitHub.fetch_pull_requests(name, tap_remote_repo, state: state, version: version)
|
||||||
|
rescue GitHub::API::ValidationFailedError => e
|
||||||
|
odebug "Error fetching pull requests for #{formula_or_cask} #{name}: #{e}"
|
||||||
|
nil
|
||||||
|
end
|
||||||
|
|
||||||
if pull_requests&.any?
|
if pull_requests&.any?
|
||||||
pull_requests = pull_requests.map { |pr| "#{pr["title"]} (#{Formatter.url(pr["html_url"])})" }.join(", ")
|
pull_requests = pull_requests.map { |pr| "#{pr["title"]} (#{Formatter.url(pr["html_url"])})" }.join(", ")
|
||||||
end
|
end
|
||||||
|
Loading…
x
Reference in New Issue
Block a user