Skip to content

Remove comment-posting from fork-PR skip, use warning annotation#49

Merged
mtfishman merged 1 commit into
mainfrom
mf/fix-fork-skip
Mar 22, 2026
Merged

Remove comment-posting from fork-PR skip, use warning annotation#49
mtfishman merged 1 commit into
mainfrom
mf/fix-fork-skip

Conversation

@mtfishman

Copy link
Copy Markdown
Member

Summary

  • Remove peter-evans/find-comment and peter-evans/create-or-update-comment steps from the fork-PR skip logic in IntegrationTest.yml
  • Fork PRs get read-only GITHUB_TOKEN, so comment-posting fails with "Resource not accessible by integration"
  • Replace with a ::warning:: annotation that is visible in the job summary
  • The skip logic itself is unchanged

Test plan

🤖 Generated with Claude Code

Fork PRs get read-only GITHUB_TOKEN so posting comments from
IntegrationTest.yml is not possible. Remove the find-comment and
create-or-update-comment steps and use a ::warning:: annotation
instead, which is visible in the job summary.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@mtfishman mtfishman merged commit f0557e7 into main Mar 22, 2026
1 check passed
@mtfishman mtfishman deleted the mf/fix-fork-skip branch March 22, 2026 19:37
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