Skip to content

ci: skip pr comment workflow on forked prs#525

Closed
bd82 wants to merge 2 commits into
mainfrom
review-pr-524
Closed

ci: skip pr comment workflow on forked prs#525
bd82 wants to merge 2 commits into
mainfrom
review-pr-524

Conversation

@bd82

@bd82 bd82 commented Jun 17, 2026

Copy link
Copy Markdown
Member

No description provided.

alex-gilin and others added 2 commits June 16, 2026 17:42
GitHub's GITHUB_TOKEN has read-only access for pull_request events from
forks, so the peter-evans/create-or-update-comment step was failing with
"Resource not accessible by integration".

Split the commenting logic into a separate workflow_run-triggered workflow
(pr-comment.yml) that runs in the base repo context where GITHUB_TOKEN
has full write access. The CI workflow now saves PR metadata (PR number,
SHA, run ID, artifact ID) as a short-lived artifact which the comment
workflow downloads and uses to post/update the PR comment.
@bd82 bd82 closed this Jun 17, 2026
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.

2 participants