Merge pull request #8169 from Rylan12/bump-formula-pr-args
bump-formula-pr: add args to check_all_pull_requests
This commit is contained in:
commit
ad4d2ac1da
@ -128,7 +128,7 @@ module Homebrew
|
|||||||
check_open_pull_requests(formula, tap_full_name, args: args)
|
check_open_pull_requests(formula, tap_full_name, args: args)
|
||||||
|
|
||||||
new_version = args.version
|
new_version = args.version
|
||||||
check_all_pull_requests(formula, tap_full_name, version: new_version) if new_version
|
check_all_pull_requests(formula, tap_full_name, version: new_version, args: args) if new_version
|
||||||
|
|
||||||
requested_spec = :stable
|
requested_spec = :stable
|
||||||
formula_spec = formula.stable
|
formula_spec = formula.stable
|
||||||
@ -159,10 +159,10 @@ module Homebrew
|
|||||||
old_version = old_formula_version.to_s
|
old_version = old_formula_version.to_s
|
||||||
forced_version = new_version.present?
|
forced_version = new_version.present?
|
||||||
new_url_hash = if new_url && new_hash
|
new_url_hash = if new_url && new_hash
|
||||||
check_all_pull_requests(formula, tap_full_name, url: new_url) unless new_version
|
check_all_pull_requests(formula, tap_full_name, url: new_url, args: args) unless new_version
|
||||||
true
|
true
|
||||||
elsif new_tag && new_revision
|
elsif new_tag && new_revision
|
||||||
check_all_pull_requests(formula, tap_full_name, url: old_url, tag: new_tag) unless new_version
|
check_all_pull_requests(formula, tap_full_name, url: old_url, tag: new_tag, args: args) unless new_version
|
||||||
false
|
false
|
||||||
elsif !hash_type
|
elsif !hash_type
|
||||||
odie "#{formula}: no --tag= or --version= argument specified!" if !new_tag && !new_version
|
odie "#{formula}: no --tag= or --version= argument specified!" if !new_tag && !new_version
|
||||||
@ -173,7 +173,7 @@ module Homebrew
|
|||||||
and old tag are both #{new_tag}.
|
and old tag are both #{new_tag}.
|
||||||
EOS
|
EOS
|
||||||
end
|
end
|
||||||
check_all_pull_requests(formula, tap_full_name, url: old_url, tag: new_tag) unless new_version
|
check_all_pull_requests(formula, tap_full_name, url: old_url, tag: new_tag, args: args) unless new_version
|
||||||
resource_path, forced_version = fetch_resource(formula, new_version, old_url, tag: new_tag)
|
resource_path, forced_version = fetch_resource(formula, new_version, old_url, tag: new_tag)
|
||||||
new_revision = Utils.popen_read("git -C \"#{resource_path}\" rev-parse -q --verify HEAD")
|
new_revision = Utils.popen_read("git -C \"#{resource_path}\" rev-parse -q --verify HEAD")
|
||||||
new_revision = new_revision.strip
|
new_revision = new_revision.strip
|
||||||
@ -190,7 +190,7 @@ module Homebrew
|
|||||||
#{new_url}
|
#{new_url}
|
||||||
EOS
|
EOS
|
||||||
end
|
end
|
||||||
check_all_pull_requests(formula, tap_full_name, url: new_url) unless new_version
|
check_all_pull_requests(formula, tap_full_name, url: new_url, args: args) unless new_version
|
||||||
resource_path, forced_version = fetch_resource(formula, new_version, new_url)
|
resource_path, forced_version = fetch_resource(formula, new_version, new_url)
|
||||||
tar_file_extensions = %w[.tar .tb2 .tbz .tbz2 .tgz .tlz .txz .tZ]
|
tar_file_extensions = %w[.tar .tb2 .tbz .tbz2 .tgz .tlz .txz .tZ]
|
||||||
if tar_file_extensions.any? { |extension| new_url.include? extension }
|
if tar_file_extensions.any? { |extension| new_url.include? extension }
|
||||||
@ -502,7 +502,7 @@ module Homebrew
|
|||||||
check_for_duplicate_pull_requests(pull_requests, args: args)
|
check_for_duplicate_pull_requests(pull_requests, args: args)
|
||||||
end
|
end
|
||||||
|
|
||||||
def check_all_pull_requests(formula, tap_full_name, version: nil, url: nil, tag: nil)
|
def check_all_pull_requests(formula, tap_full_name, version: nil, url: nil, tag: nil, args:)
|
||||||
unless version
|
unless version
|
||||||
specs = {}
|
specs = {}
|
||||||
specs[:tag] = tag if tag
|
specs[:tag] = tag if tag
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user