Skip to content

fix goreleaser ignore_tags pattern to use glob syntax#725

Merged
aviator-app[bot] merged 1 commit intomasterfrom
fix-goreleaser-ignore-tags-glob
Apr 22, 2026
Merged

fix goreleaser ignore_tags pattern to use glob syntax#725
aviator-app[bot] merged 1 commit intomasterfrom
fix-goreleaser-ignore-tags-glob

Conversation

@tulioz
Copy link
Copy Markdown
Contributor

@tulioz tulioz commented Apr 22, 2026

goreleaser passes ignore_tags to git describe --exclude which expects glob, not regex

@tulioz tulioz requested a review from a team as a code owner April 22, 2026 19:15
@aviator-app
Copy link
Copy Markdown
Contributor

aviator-app Bot commented Apr 22, 2026

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR was merged using Aviator.


See the real-time status of this PR on the Aviator webapp.
Use the Aviator Chrome Extension to see the status of your PR within GitHub.

@aviator-app
Copy link
Copy Markdown
Contributor

aviator-app Bot commented Apr 22, 2026

✅ FlexReview Status

Common Owner: aviator-co/engineering (expert-load-balance assignment)
Owner and Assignment:

  • aviator-co/engineering (expert-load-balance assignment)
    Owned Files
    • 🔒 .goreleaser.yaml

Review SLO: 7 business hours if PR size is <= 200 LOC for the first response.

@aviator-app aviator-app Bot requested a review from brain-crystal April 22, 2026 19:15
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the git ignore_tags pattern in .goreleaser.yaml to use glob syntax. The feedback suggests extending the pattern to include a trailing asterisk to ensure that nightly tags with suffixes are correctly captured and ignored.

Comment thread .goreleaser.yaml
git:
ignore_tags:
- ".*-nightly"
- "*-nightly"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The change to glob syntax is correct. However, consider using *-nightly* (adding a trailing asterisk) to ensure that nightly tags with suffixes (e.g., v1.0.0-nightly.20231027 or build numbers) are also correctly ignored. The current pattern *-nightly only matches tags that end exactly with that string.

    - "*-nightly*"

@aviator-app aviator-app Bot merged commit f2effbe into master Apr 22, 2026
5 checks passed
@aviator-app aviator-app Bot deleted the fix-goreleaser-ignore-tags-glob branch April 22, 2026 19:17
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