Skip to content

Let the source feed update itself when a release is published - #28

Merged
g8row merged 1 commit into
mainfrom
fix/source-feed-on-release
Sep 20, 2026
Merged

g8row merged 1 commit into
mainfrom
fix/source-feed-on-release

Conversation

@g8row

@g8row g8row commented Sep 20, 2026

Copy link
Copy Markdown
Owner

Found while publishing 0.3.7.

update-source has only ever worked from a manual run. On a release event actions/checkout takes the tag, so the job built docs/apps.json, committed it, and then could not push:

[detached HEAD 6c07a89] source: update apps.json
 1 file changed, 6 insertions(+), 6 deletions(-)
fatal: You are not currently on a branch.

That is run 35521862717 on the 0.3.7 release — the feed stayed on 0.3.6 with the release already published. It predates #10's merge into a release flow; the apps.json commits on main so far came from manual dispatches, which check out a branch and so push fine.

The job never needs the tagged tree — it rebuilds the feed from the GitHub API — so it now checks out the default branch, where the commit belongs, and pushes there explicitly.

Verified after merge by re-running the workflow against the published 0.3.7 release.

🤖 Generated with Claude Code

The workflow has only ever worked from a manual run. On a release event
`actions/checkout` takes the tag, so the job built `docs/apps.json`,
committed it, and then could not push it: "fatal: You are not currently on
a branch". Publishing 0.3.7 hit exactly that, leaving the feed pointing at
0.3.6 with the release already out.

It now checks out the default branch, which is where the commit belongs —
the feed is rebuilt from the GitHub API, never from the tree — and pushes
there explicitly.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@g8row
g8row merged commit a99d286 into main Sep 20, 2026
1 check passed
@g8row
g8row deleted the fix/source-feed-on-release branch September 20, 2026 16:25
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