Skip to content

check-for-epoch-bump: also compare versions#28

Merged
xnox merged 1 commit into
chainguard-dev:mainfrom
xnox:xnox/os-521-fix-epoch-bump-pre-commit-to-support-version-bumps
Feb 19, 2026
Merged

check-for-epoch-bump: also compare versions#28
xnox merged 1 commit into
chainguard-dev:mainfrom
xnox:xnox/os-521-fix-epoch-bump-pre-commit-to-support-version-bumps

Conversation

@xnox

@xnox xnox commented Feb 18, 2026

Copy link
Copy Markdown
Member

When working on new upstream releases check-for-epoch-bump does not
believe that new upstream releases has higher full-version.

When working on new upstream releases check-for-epoch-bump does not
believe that new upstream releases has higher full-version.
@xnox xnox requested a review from a team February 18, 2026 22:51

@EyeCantCU EyeCantCU left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is definitely an improvement, but I have a couple suggestions

Comment thread scripts/check-for-epoch-bump.sh
Comment thread scripts/check-for-epoch-bump.sh

@EyeCantCU EyeCantCU left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for improving the check, this is definitely better. Suggestions are more nits than anything

@xnox

xnox commented Feb 19, 2026

Copy link
Copy Markdown
Member Author

Thanks for improving the check, this is definitely better. Suggestions are more nits than anything

Will follow up in #29

@xnox xnox merged commit c888286 into chainguard-dev:main Feb 19, 2026
4 checks passed
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