Skip to content

ci: use a PAT for release-please and post manifest diff comments - #15

Merged
perfectra1n merged 2 commits into
mainfrom
ci/release-please-pat
Jul 22, 2026
Merged

ci: use a PAT for release-please and post manifest diff comments#15
perfectra1n merged 2 commits into
mainfrom
ci/release-please-pat

Conversation

@perfectra1n

Copy link
Copy Markdown
Member

Why

Two CI fixes from the #14 branch were pushed a few minutes after that PR was merged, so they never landed on main:

  1. The post-merge Release Please run failed with "GitHub Actions is not permitted to create or approve pull requests" because the org policy blocks the built-in GITHUB_TOKEN from creating PRs. The workflow now uses the RELEASE_PLEASE_TOKEN secret (already configured in the repo) with a fallback to GITHUB_TOKEN.
  2. The manifest diff comment never appeared on PRs: diff-action v2 only formats the diff into an output, so int128/comment-action is needed to actually post it.

After merging

Trigger the Release Please workflow once (it has workflow_dispatch) or wait for the next push to main. It will open the "release trilium 2.0.0" PR using the PAT, with the changelog built from the feat! commit in #14.

diff-action v2 only formats the diff and sets an output; it no longer
posts the comment itself. Post it with int128/comment-action, replacing
the previous comment on each push.
The org policy does not allow the built-in GITHUB_TOKEN to create pull
requests, and a PAT-created release PR also gets normal PR validation
(GITHUB_TOKEN events never trigger other workflows). Falls back to
GITHUB_TOKEN when the RELEASE_PLEASE_TOKEN secret is not configured.
@gemini-code-assist

Copy link
Copy Markdown

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@perfectra1n
perfectra1n merged commit 4335635 into main Jul 22, 2026
3 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.

1 participant