Skip to content

Comments

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

Merged
xnox merged 1 commit intochainguard-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 intochainguard-dev:mainfrom
xnox:xnox/os-521-fix-epoch-bump-pre-commit-to-support-version-bumps

Conversation

@xnox
Copy link
Member

@xnox xnox commented Feb 18, 2026

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
Copy link
Member

@EyeCantCU EyeCantCU left a comment

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

Copy link
Member

@EyeCantCU EyeCantCU left a comment

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
Copy link
Member Author

xnox commented Feb 19, 2026

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