Skip to content

fix: correct version ordering in versionChanges#2633

Closed
maphew wants to merge 1 commit into
gastownhall:mainfrom
maphew:fix-version-order
Closed

fix: correct version ordering in versionChanges#2633
maphew wants to merge 1 commit into
gastownhall:mainfrom
maphew:fix-version-order

Conversation

@maphew

@maphew maphew commented Mar 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Move v0.61.0 entry to beginning of versionChanges slice for proper newest-first ordering
  • Fixes TestGetVersionsSinceOrder failure where 0.61.0 appeared after older versions

@DreadPirateRobertz DreadPirateRobertz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Correct fix — moves v0.61.0 to the top of the slice for proper newest-first ordering. Also removes the duplicate v0.60.0 entry that was left behind. LGTM.

steveyegge added a commit that referenced this pull request Mar 17, 2026
Move v0.61.0 entry to the beginning for proper newest-first ordering and
remove duplicate v0.60.0 entry. Fixes TestGetVersionsSinceOrder.

Co-Authored-By: maphew <maphew@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@steveyegge

Copy link
Copy Markdown
Contributor

Fix-merged to main in 0e13cb5. Thank you @maphew!

@steveyegge steveyegge closed this Mar 17, 2026
@maphew maphew deleted the fix-version-order branch March 17, 2026 15:21
julianknutsen pushed a commit to gascity/beads-fork that referenced this pull request Mar 22, 2026
Move v0.61.0 entry to the beginning for proper newest-first ordering and
remove duplicate v0.60.0 entry. Fixes TestGetVersionsSinceOrder.

Co-Authored-By: maphew <maphew@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

3 participants