Skip to content

ci(release): retry the tap dispatch and say what to do when it fails - #2

Merged
noidee-dev merged 1 commit into
mainfrom
fix/tap-bump-resilience
Sep 23, 2026
Merged

noidee-dev merged 1 commit into
mainfrom
fix/tap-bump-resilience

Conversation

@noidee-dev

Copy link
Copy Markdown
Contributor

The tap job runs after the release is published. A failure there is not "a build that did not happen" — it is a live tap pointing at the previous version while its README advertises this one. v1.0.0 did exactly that.

  • Three dispatch attempts, for a transient API error.
  • An if: failure() step that prints the ready-to-paste gh workflow run command with the real version and checksum filled in, and names the likely cause. A bare 403 in a red job says nothing about what to do next.

What retrying cannot fix is a token that is not permitted to dispatch, which is what actually happened — TAP_TOKEN still returns HTTP 403 after three attempts at adjusting it, and the same 403 hit dotfix on 2026-09-22. The companion change is homebrew-tap#2, which verifies the artifact before committing so a failed bump leaves the tap on its previous working version.

Not exercised yet: the failure path only runs on a real release, and gh run rerun replays the old workflow definition. The message rendering was checked locally; the retry loop is three lines around the unchanged command.

🤖 Generated with Claude Code

The tap job runs after the release is already published, so a failure
there is not a build that did not happen -- it is a live tap pointing at
the previous version while the README advertises this one. That is
exactly what v1.0.0 did.

Three attempts covers a transient API error. What it cannot fix is a
token that is not allowed to dispatch, which is what actually happened,
so the failure path now prints the ready-to-paste command and names the
likely cause instead of leaving a red job with a bare 403 in it.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@noidee-dev
noidee-dev merged commit c366f84 into main Sep 23, 2026
2 checks passed
@noidee-dev
noidee-dev deleted the fix/tap-bump-resilience branch September 23, 2026 12:13
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.

1 participant