Skip to content

fix: pass GH_TOKEN to deploy-shared.yml's deploy step - #141

Merged
ineedjet merged 1 commit into
mainfrom
fix-deploy-gh-token
Aug 24, 2026
Merged

fix: pass GH_TOKEN to deploy-shared.yml's deploy step#141
ineedjet merged 1 commit into
mainfrom
fix-deploy-gh-token

Conversation

@ineedjet

Copy link
Copy Markdown
Collaborator

Summary

First real deploy attempt (to heimdall, triggered by the v0.6.0 release push) failed at ref resolution:

resolve.RefError: Could not resolve GitHub latest release for rubykatzen/flightdeck: gh: To use GitHub CLI in a GitHub Actions workflow, set the GH_TOKEN environment variable.

deploy/deploy.py shells out to the gh CLI (via deploy/resolve.py) to resolve @latest and download release assets. Every other job that needs gh (upload-apps, encrypt) already passes a token explicitly to its action; the deploy step in deploy-shared.yml never did. Fixed by adding GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} to that step's env, alongside the existing SOPS_AGE_KEY.

Test plan

  • pre-commit (yamllint, actionlint) passes
  • Next release/manual dispatch actually reaches heimdall

deploy/deploy.py resolves and downloads release refs via the gh CLI
(deploy/resolve.py), which needs GH_TOKEN/GITHUB_TOKEN in its
environment. Every other job that shells out to gh (upload-apps,
encrypt) already passes a token explicitly to its action; the deploy
step never did, so every real deploy failed at ref resolution:
"Could not resolve GitHub latest release ... set the GH_TOKEN
environment variable."

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@ineedjet
ineedjet merged commit 4619a8d into main Aug 24, 2026
5 checks passed
@ineedjet
ineedjet deleted the fix-deploy-gh-token branch August 24, 2026 13: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.

1 participant