ci: skip gRPC coverage PR comment on fork PRs#3136
ci: skip gRPC coverage PR comment on fork PRs#3136PastaPastaPasta merged 1 commit intodashpay:v3.1-devfrom
Conversation
Fork PRs don't have write permissions for the GITHUB_TOKEN, so the Comment PR step fails with 'Resource not accessible by integration'. Skip the comment step when the PR head repo differs from the base repo. The coverage report is still available as a build artifact.
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
PastaPastaPasta
left a comment
There was a problem hiding this comment.
utACK; simple CI change to unblock external PRs
Option B (alternative to #3135)
Skip the Comment PR step when the PR comes from a fork (
head.repo.full_name != repository). Fork PRs don't get write permissions forGITHUB_TOKEN, causing the comment step to fail withResource not accessible by integration.The coverage check itself still runs and fails/passes normally. The report is available as a build artifact. Comments still post on internal PRs.