Skip to content

fix: use GitHub App token for docs update PRs#1145

Merged
amisevsk merged 1 commit into
kitops-ml:mainfrom
gorkem:fix/docs-pr-app-token
Apr 2, 2026
Merged

fix: use GitHub App token for docs update PRs#1145
amisevsk merged 1 commit into
kitops-ml:mainfrom
gorkem:fix/docs-pr-app-token

Conversation

@gorkem

@gorkem gorkem commented Mar 29, 2026

Copy link
Copy Markdown
Member

What changed

Use the existing KITOPS_BOT GitHub App token instead of the default GITHUB_TOKEN when creating docs update PRs in the release workflow.

Why

PRs created with the default GITHUB_TOKEN don't trigger downstream workflows (like check-sources) due to a GitHub limitation. This blocks docs update PRs from being merged without manual intervention (e.g. rebasing).

The publish-homebrew-tap-formula job already uses the app token for this reason — this applies the same pattern to the docs update step.

Validation

  • YAML syntax validated
  • Follows the same token generation pattern already used in publish-homebrew-tap-formula

Fixes #750

@gorkem gorkem force-pushed the fix/docs-pr-app-token branch from 5c66797 to 7ed0af4 Compare March 29, 2026 02:35
@gorkem gorkem requested review from amisevsk and Copilot and removed request for Copilot March 29, 2026 02:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the release workflow to authenticate docs-update PR creation with the KITOPS_BOT GitHub App token so downstream workflows (e.g., check-sources) are triggered.

Changes:

  • Adds a step to mint a GitHub App installation token (KITOPS_BOT) in the release job.
  • Uses the app token for the docs update git/gh operations instead of the default GITHUB_TOKEN.
  • Updates the git remote to push using the app token.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/platform-release.yaml Outdated
Use the existing KITOPS_BOT GitHub App token instead of the default
GITHUB_TOKEN when creating docs update PRs during release. The default
token does not trigger downstream workflows (e.g. check-sources),
which blocks these PRs from being merged without manual intervention.

Fixes kitops-ml#750

Signed-off-by: Gorkem Ercan <gorkem.ercan@gmail.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@amisevsk amisevsk merged commit 2ec8244 into kitops-ml:main Apr 2, 2026
12 checks passed
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.

Docs update PRs do not trigger check-sources actions and so cannot be merged

3 participants