Skip to content

fix stable release changelog to include all changes since last stable#716

Merged
aviator-app[bot] merged 1 commit intomasterfrom
fix-stable-release-changelog
Apr 14, 2026
Merged

fix stable release changelog to include all changes since last stable#716
aviator-app[bot] merged 1 commit intomasterfrom
fix-stable-release-changelog

Conversation

@tulioz
Copy link
Copy Markdown
Contributor

@tulioz tulioz commented Apr 14, 2026

goreleaser was picking the nightly tag as the previous tag, so stable releases only showed incremental changes instead of the full changelog since last stable.

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

aviator-app Bot commented Apr 14, 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 14, 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
    • 🔒 .github/workflows/nightly-release.yml

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 14, 2026 22:50
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 .goreleaser.yaml configuration to ignore nightly tags during the release process. The review feedback suggests expanding the tag exclusion regex to include other pre-release identifiers like RC, beta, and alpha to ensure that stable release changelogs are correctly generated relative to the last stable version.

Comment thread .goreleaser.yaml

git:
ignore_tags:
- ".*-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 current regex .*-nightly only filters out tags containing -nightly. If the project uses other pre-release identifiers (such as -rc, -beta, or -alpha), those tags will still be picked up as the 'previous tag', causing the stable release changelog to be incremental rather than since the last stable version. To fully satisfy the goal of including all changes since the last stable release, consider using a more inclusive regex like .*-(nightly|rc|beta|alpha).* or .*-.* (if all pre-releases contain a hyphen). Additionally, note that this setting is global and will cause nightly releases to also show a full changelog since the last stable release, rather than just the changes since the previous nightly.

    - ".*-(nightly|rc|beta|alpha).*"

@aviator-app aviator-app Bot merged commit a5cbc55 into master Apr 14, 2026
5 checks passed
@aviator-app aviator-app Bot deleted the fix-stable-release-changelog branch April 14, 2026 22:54
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