Skip to content

fix: handle fork PR permission errors in review bot#992

Open
max-tech-bot wants to merge 1 commit intoDFXswiss:developfrom
max-tech-bot:fix/pr-review-bot-permissions
Open

fix: handle fork PR permission errors in review bot#992
max-tech-bot wants to merge 1 commit intoDFXswiss:developfrom
max-tech-bot:fix/pr-review-bot-permissions

Conversation

@max-tech-bot
Copy link
Contributor

The GITHUB_TOKEN has limited permissions for fork PRs, causing the Post PR comment step to fail with 403 Resource not accessible by integration.

Changes:

  • Added continue-on-error: true to the Post PR comment step so the job doesn't fail
  • Wrapped the comment posting API calls in try/catch with a warning message
  • The review checks (ESLint, TypeScript, audit, TODOs) still run and report correctly — only the comment posting is gracefully skipped when permissions are insufficient

The GITHUB_TOKEN has limited permissions for fork PRs, causing the
Post PR comment step to fail with 403. Added continue-on-error and
try/catch to handle this gracefully instead of failing the job.
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