Skip to content

fix: skip Homebrew tap update when releasing older patch versions#4224

Open
alonkeyval wants to merge 2 commits intoodigos-io:mainfrom
alonkeyval:alon/plat-754-updates-the-homebrew-tap-if-the-new-version-is-greater-than
Open

fix: skip Homebrew tap update when releasing older patch versions#4224
alonkeyval wants to merge 2 commits intoodigos-io:mainfrom
alonkeyval:alon/plat-754-updates-the-homebrew-tap-if-the-new-version-is-greater-than

Conversation

@alonkeyval
Copy link
Collaborator

Adds a semver comparison check before GoReleaser publishes the Homebrew formula. This prevents older patch releases (e.g. v1.16.9) from overwriting a newer version (e.g. v1.17.0) in the homebrew-odigos-cli tap.

alonbraymok and others added 2 commits February 10, 2026 15:17
Adds a semver comparison check before GoReleaser publishes the Homebrew
formula. This prevents older patch releases (e.g. v1.16.9) from
overwriting a newer version (e.g. v1.17.0) in the homebrew-odigos-cli tap.

Co-authored-by: Cursor <cursoragent@cursor.com>
fi

# Strip 'v' prefix from release tag for comparison
RELEASE_VERSION="${TAG#v}"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this work with -rc0 & -pre0 ?

owner: odigos-io
name: '{{ if eq .Env.IS_PRERELEASE "true" }}homebrew-odigos-cli-rc{{ else }}homebrew-odigos-cli{{ end }}'
token: "{{ .Env.HOMEBREW_GITHUB_API_TOKEN }}"
skip_upload: '{{ .Env.SKIP_HOMEBREW_UPDATE }}'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We do want to upload but not overwrite the latest version. Is this what this is doing - or is skipping the upload completely?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alonbraymok the same thing I told u

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants