Skip to content

fix(ci): inherit secrets into the release workflow, serialize tag-on-main - #175

Merged
thetillhoff merged 1 commit into
mainfrom
fix/inherit-secrets-and-serialize
Jul 31, 2026
Merged

fix(ci): inherit secrets into the release workflow, serialize tag-on-main#175
thetillhoff merged 1 commit into
mainfrom
fix/inherit-secrets-and-serialize

Conversation

@thetillhoff

Copy link
Copy Markdown
Owner

Observed on temingo v3.0.5: the release succeeded, the run went red, because the homebrew tap step got an empty GH_TOKEN. A called workflow receives no secrets unless the caller passes them, so secrets.HOMEBREW_TAP_ACCESS_TOKEN resolves to empty on the workflow_call path. Third instance of that path diverging from on: push: tags: (see #174 for the first two). secrets: inherit fixes it.

Also adds a concurrency group to tag-on-main. Two renovate merges close together run in parallel, both read the same git describe output, and compute the same next version - one tag push wins, the other fails. cancel-in-progress: false, since cancelling mid-run is how you end up with a tag whose release never happened.

Counterpart: thetillhoff/temingo#100.

…main

Observed on temingo v3.0.5: release succeeded, run went red, because the
homebrew tap step got an empty GH_TOKEN. A called workflow receives no
secrets unless the caller passes them. Third instance of the workflow_call
path diverging from `on: push: tags:`.

Also adds a concurrency group. Two merges in quick succession would run in
parallel, both read the same `git describe` tag and compute the same next
version - one tag push wins, the other fails. Not cancel-in-progress: a
cancelled run leaves a tag whose release never happened.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@thetillhoff
thetillhoff enabled auto-merge July 31, 2026 18:24
@thetillhoff
thetillhoff merged commit 4070382 into main Jul 31, 2026
8 checks passed
@thetillhoff
thetillhoff deleted the fix/inherit-secrets-and-serialize branch July 31, 2026 18:26
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