admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

bump-cask-pr: fix duplicate PR checking with comma versions

Browse Source
This commit is contained in:
Bo Anderson 2023-11-30 02:49:07 +00:00
parent 8b2af53f32
commit 0aae1fa6ae
No known key found for this signature in database
GPG Key ID: 3DB94E204E137D65
1 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
Library/Homebrew/dev-cmd/bump-cask-pr.rb
View File

@ -139,11 +139,7 @@ module Homebrew
# For simplicity, our naming defers to the arm version if we multiple architectures are specified
branch_version = new_version.arm || new_version.general
if branch_version.is_a?(Cask::DSL::Version)
commit_version = if branch_version.before_comma == cask.version.before_comma
branch_version
else
branch_version.before_comma
end
commit_version = shortened_version(branch_version, cask: cask)
branch_name = "bump-#{cask.token}-#{branch_version.tr(",:", "-")}"
commit_message ||= "#{cask.token} #{commit_version}"
end
@ -174,6 +170,15 @@ module Homebrew
GitHub.create_bump_pr(pr_info, args: args)
end
sig { params(version: Cask::DSL::Version, cask: Cask::Cask).returns(Cask::DSL::Version) }
def shortened_version(version, cask:)
if version.before_comma == cask.version.before_comma
version
else
version.before_comma
end
end
sig {
params(
cask: Cask::Cask,
@ -260,7 +265,7 @@ module Homebrew
cask.token,
tap_remote_repo,
state: "closed",
version: version,
version: shortened_version(version, cask: cask),
file: cask.sourcefile_path.relative_path_from(cask.tap.path).to_s,
args: args,
)