Skip to content

fix: simplify release workflow, remove GITHUB_TOKEN workarounds#17

Merged
GraysonCAdams merged 1 commit intomainfrom
fix/simplify-release-workflow
Mar 1, 2026
Merged

fix: simplify release workflow, remove GITHUB_TOKEN workarounds#17
GraysonCAdams merged 1 commit intomainfrom
fix/simplify-release-workflow

Conversation

@GraysonCAdams
Copy link
Copy Markdown
Collaborator

Summary

  • Strip all anti-pattern workarounds from release workflow (fake status checks, inline CodeQL, inline Docker)
  • Simplify release.yml to just release-please (~15 lines)
  • Restore docker-publish.yml with both release: published and workflow_dispatch triggers
  • Add TODO documenting the proper fix: GitHub App token via actions/create-github-app-token

Why

The previous iterations layered workarounds for GITHUB_TOKEN not triggering workflows. The right fix is a GitHub App token, which makes everything work naturally. Until that's set up, releases and Docker publishes can be done manually.

Test plan

  • Verify release.yml runs cleanly on push to main
  • Verify docker-publish.yml can be triggered via workflow_dispatch

Strip out anti-patterns from the release workflow:
- Remove fake status check injection via commit status API
- Remove inline CodeQL job for release PRs
- Remove inline Docker build job

These were workarounds for GITHUB_TOKEN not triggering other workflows.
The proper fix is a GitHub App token (documented in TODO comment).

Restore docker-publish.yml with both release event trigger and manual
workflow_dispatch fallback, with proper version extraction for
release-please's scrolly-v* tag format.
@GraysonCAdams GraysonCAdams merged commit f88d1f8 into main Mar 1, 2026
21 checks passed
@GraysonCAdams GraysonCAdams deleted the fix/simplify-release-workflow branch March 1, 2026 06:19
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