Skip to content

Conversation

@schgoo
Copy link
Collaborator

@schgoo schgoo commented Jan 16, 2026

Fork PRs have read-only GITHUB_TOKEN and cannot post comments. Trying to see if we can skip the sticky-pull-request-comment steps when the PR comes from a fork to avoid "Resource not accessible by integration" errors.

@codecov
Copy link

codecov bot commented Jan 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.0%. Comparing base (061e578) to head (812bd89).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #205   +/-   ##
=======================================
  Coverage   100.0%   100.0%           
=======================================
  Files         105      105           
  Lines        6761     6761           
=======================================
  Hits         6761     6761           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@schgoo schgoo changed the title ci: skip PR comment steps for fork PRs ci: Fix semver PR comment check for fork PRs Jan 17, 2026
@geeknoid
Copy link
Member

So PRs from a fork won't get notified about semver breaks?

@schgoo
Copy link
Collaborator Author

schgoo commented Jan 20, 2026

So PRs from a fork won't get notified about semver breaks?

@geeknoid This is already true; the semver check fails regardless of semver breaks, and no comments are made, because the GITHUB-TOKEN for fork PRs has read-only access. I'm open to a better suggestion, though. For example, we could:

  1. Also remove continue-on-error: true for the Semver Compatibility check, so it actually fails if the semver check fails.
  2. Allow us to check in code to non-main branches.
  3. Upload semver results as an artifact, and have a separate oxidizer repo-local workflow that triggers the PR comments (not sure how you'd do comment cleanup there, though).

Fork PRs have read-only GITHUB_TOKEN and cannot post comments.
Skip the sticky-pull-request-comment steps when the PR comes from
a fork to avoid "Resource not accessible by integration" errors.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@geeknoid geeknoid enabled auto-merge (squash) January 22, 2026 16:35
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.

3 participants