Skip to content

version-guard: fetch the last-cut tag with a destination refspec - #234

Merged
lannbot merged 1 commit into
mainfrom
cut-goldens-tag-fetch
Aug 23, 2026
Merged

version-guard: fetch the last-cut tag with a destination refspec#234
lannbot merged 1 commit into
mainfrom
cut-goldens-tag-fetch

Conversation

@lannbot

@lannbot lannbot commented Aug 23, 2026

Copy link
Copy Markdown
Collaborator

First v0.5.0 cut dispatch failed closed in cut-conventions-goldens: the tag fetch lacked a destination refspec, so the shallow release checkout never got a resolvable v0.4.0 ref (fatal: bad revision). Objects-into-FETCH_HEAD suffices for the sha-addressed PR-base fetch this mirrored; a tag name needs a local ref.

Fix: git fetch origin +refs/tags/v0.4.0:refs/tags/v0.4.0 --depth=1. Verified against a real --depth=1 tag-less clone: old form keeps failing, new form resolves and returns the window's all-A golden statuses (adds are free — the guard then passes). Mock argv keys re-pinned; just test-version-guard 50/50.

Unblocks the v0.5.0 cut re-dispatch.

The v0.5.0 cut's first dispatch failed (closed, as designed) in
cut-conventions-goldens: 'git fetch origin refs/tags/v0.4.0' drops the
objects into FETCH_HEAD but creates no local ref, so the tag NAME stays
unresolvable in the release job's shallow checkout and the goldens diff
dies with 'bad revision'. The PR-base fetch this mirrored gets away with
a bare source because a raw sha resolves from the object store alone; a
tag name needs a ref. Fetch +refs/tags/X:refs/tags/X instead; verified
against a real shallow tag-less clone (mock argv keys are re-pinned, but
they are what missed this).
@lannbot
lannbot enabled auto-merge August 23, 2026 02:41
@lannbot
lannbot merged commit 408b803 into main Aug 23, 2026
4 checks passed
@lannbot
lannbot deleted the cut-goldens-tag-fetch branch August 23, 2026 16:50
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