Skip to content

fix: use published_at for release sorting#46

Merged
igorpecovnik merged 1 commit intomainfrom
fix
Jan 20, 2026
Merged

fix: use published_at for release sorting#46
igorpecovnik merged 1 commit intomainfrom
fix

Conversation

@igorpecovnik
Copy link
Copy Markdown
Member

@igorpecovnik igorpecovnik commented Jan 20, 2026

Summary

  • Fix release sorting to use published_at instead of created_at
  • Prevents accidental deletion of newer releases when multiple releases have identical creation timestamps

Problem

When releases are created in bulk (e.g., multiple trunk releases created simultaneously), they share the same created_at timestamp. This causes unstable sort order in the delete-old-releases workflow, resulting in newer releases being deleted while older ones are kept.

Solution

Changed sorting from created_at to published_at, which reflects the actual publication order regardless of when the release object was initially created.

Test plan

  • Manual verification with dry-run shows correct releases selected for deletion
  • Newest releases (by publish date) are preserved
  • Older incomplete prereleases are properly cleaned up

When multiple releases are created in bulk, they have identical created_at
timestamps, causing unstable sort order that may delete newer releases
instead of older ones. Use published_at which reflects the actual release
publication order.
@igorpecovnik igorpecovnik merged commit e901ef0 into main Jan 20, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant