Skip to content

fix: Accept major.minor version in package-for-homebrew.sh#11

Merged
GordonBeeming merged 2 commits intomainfrom
gb/fix-version-validation
Apr 2, 2026
Merged

fix: Accept major.minor version in package-for-homebrew.sh#11
GordonBeeming merged 2 commits intomainfrom
gb/fix-version-validation

Conversation

@GordonBeeming
Copy link
Copy Markdown
Owner

Summary

  • Version validation in package-for-homebrew.sh rejected 0.3 (required 3-part semver)
  • Now accepts both major.minor and major.minor.patch formats

Test plan

  • Release pipeline passes with tag v0.3

Version validation rejected "0.3" — now accepts both major.minor
and major.minor.patch formats.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: GitButler <gitbutler@gitbutler.com>
Copilot AI review requested due to automatic review settings April 2, 2026 12:01
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Homebrew packaging helper to accept release tags that use a 2-part version (major.minor) in addition to full 3-part semver, aligning the script with tags like v0.3 used in the release pipeline.

Changes:

  • Relax version validation in package-for-homebrew.sh to allow major.minor and major.minor.patch.
  • Update the validation error message to reflect the newly accepted version formats.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

We don't use prereleases — keep regex and error message consistent.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: GitButler <gitbutler@gitbutler.com>
@GordonBeeming GordonBeeming merged commit cb03519 into main Apr 2, 2026
2 checks passed
@GordonBeeming GordonBeeming deleted the gb/fix-version-validation branch April 2, 2026 12:10
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.

2 participants