Skip to content

fix: allow check-warp-deploy to run on fork PRs#1373

Merged
paulbalaji merged 3 commits intomainfrom
pb/3p-checkwarpdeploy
Feb 2, 2026
Merged

fix: allow check-warp-deploy to run on fork PRs#1373
paulbalaji merged 3 commits intomainfrom
pb/3p-checkwarpdeploy

Conversation

@paulbalaji
Copy link
Copy Markdown
Collaborator

@paulbalaji paulbalaji commented Feb 2, 2026

Summary

  • Skip GitHub App token generation for fork PRs (they don't have access to secrets)
  • Write summary to GITHUB_STEP_SUMMARY so it's visible on workflow run page for all PRs
  • PR comments still posted for non-fork PRs

Fixes: https://github.com/hyperlane-xyz/hyperlane-registry/actions/runs/21583871563/job/62225555469

Test plan

  • Verify workflow passes on this PR (non-fork)
  • Re-run the failed fork PR to confirm it works
  • Check workflow summary appears on run page

🤖 Generated with Claude Code

paulbalaji and others added 2 commits February 2, 2026 15:05
Skip GitHub App token generation for fork PRs since they don't have
access to secrets. Checks still run, just no PR comment posted.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Always write the check summary to the workflow run page so it's visible
even for fork PRs that can't post PR comments.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 2, 2026

Warning

Rate limit exceeded

@paulbalaji has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 56 minutes and 24 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch pb/3p-checkwarpdeploy

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@paulbalaji paulbalaji enabled auto-merge February 2, 2026 15:09
@paulbalaji paulbalaji added this pull request to the merge queue Feb 2, 2026
Merged via the queue into main with commit 1c02368 Feb 2, 2026
12 checks passed
@paulbalaji paulbalaji deleted the pb/3p-checkwarpdeploy branch February 2, 2026 15:11
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