SisyphusMD/project-standard
Watch
0
0
Fork
You've already forked project-standard
 0
Code Issues 1 Pull requests Releases Packages Activity Actions

fix(release): compare the mirrored tag object, since a reused tag name answers 200 while stale #13

Merged
Cody Bryant merged 1 commit from fix/compare-the-mirrored-tag-object into main 2026-08-29 13:57:36 -07:00
Conversation 0 Commits 1 Files changed 2 +48 -20
Cody Bryant commented 2026-08-29 13:56:47 -07:00
Owner
Copy link

The confirmation added in v0.1.22 checked only that GitHub answered 200 for the tag name. An rc number is derived from the source tags alone, so a name can be reused, and a ref left behind by a partial prune answers 200 while still pointing at the previous release - every job waiting on the mirror would then run against the wrong commit.

Compares the object the mirror serves with the one just pushed. The test gains that case.

The confirmation added in v0.1.22 checked only that GitHub answered 200 for the tag name. An rc number is derived from the source tags alone, so a name can be reused, and a ref left behind by a partial prune answers 200 while still pointing at the previous release - every job waiting on the mirror would then run against the wrong commit. Compares the object the mirror serves with the one just pushed. The test gains that case.
fix(release): compare the mirrored tag object, since a reused tag name answers 200 while stale
All checks were successful
CI / Shell scripts (pull_request) Successful in 3s
Details
CI / The standard can actually be vendored (pull_request) Successful in 14s
Details
CI / Lint and test the standard's own code (pull_request) Successful in 22s
Details
e6bce24b67
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
SisyphusMD/project-standard!13
No description provided.