Skip to content

ci/manifest: pass --repo to gh so script doesn't need cwd .git#2113

Merged
widgetii merged 1 commit into
masterfrom
ci/manifest-fix-repo-flag
May 20, 2026
Merged

ci/manifest: pass --repo to gh so script doesn't need cwd .git#2113
widgetii merged 1 commit into
masterfrom
ci/manifest-fix-repo-flag

Conversation

@widgetii
Copy link
Copy Markdown
Member

Summary

Hotfix on top of #2112. The manifest workflow checks out master into ./master/ and gh-pages into ./pages/ — neither is the workflow's cwd, so `gh release list` fails with "fatal: not a git repository" because it tries to infer the repo from `.git` in cwd.

Fix: thread `--repo $GITHUB_REPOSITORY` into every `gh` invocation. The script is now cwd-independent.

Repro / verification

Failed run before fix: https://github.com/OpenIPC/firmware/actions/runs/26180862470

Local repro (from /tmp, no .git in cwd) produced the exact same trace. After the fix the same command emits the expected empty manifest with `# No nightly-YYYYMMDD- releases yet`.

Test plan

  • CI passes on this PR.
  • After merge, manually dispatch `manifest.yml`; expect green run and empty `manifest.json` / `manifest.flat` committed to gh-pages.
  • Tonight's first cron after merge: full pipeline lights up.

🤖 Generated with Claude Code

The manifest workflow checks out master into ./master/ and gh-pages
into ./pages/ so neither is the workflow's cwd. `gh release list`
without --repo then fails with "not a git repository". Pass --repo
$GITHUB_REPOSITORY to every gh call so the script is cwd-independent.

Reproduced locally with cd /tmp && python3 .../enrich_manifest.py /tmp/m
— same trace as the failed workflow run 26180862470. After the fix
the same command emits the expected empty manifest.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@widgetii widgetii merged commit a95d789 into master May 20, 2026
93 checks passed
@widgetii widgetii deleted the ci/manifest-fix-repo-flag branch May 20, 2026 18:39
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