Skip to content

Merge upstream: Canonicalize PR number references (#1749)#50

Merged
aaditagrawal merged 1 commit intomainfrom
merge/upstream-1749-canonicalize-pr-refs
Apr 5, 2026
Merged

Merge upstream: Canonicalize PR number references (#1749)#50
aaditagrawal merged 1 commit intomainfrom
merge/upstream-1749-canonicalize-pr-refs

Conversation

@aaditagrawal
Copy link
Copy Markdown
Owner

What

Cherry-picks upstream commit cd7980b4 (PR #1749) onto the fork.

Upstream changes

Canonicalizes PR number references by stripping the # prefix before parsing, so both #123 and 123 are handled consistently in normalizePullRequestReference.

Conflict resolution

None -- clean cherry-pick with no conflicts (2 files, +3/-3).

Verification

  • bun typecheck passes (all 7 packages)
  • pullRequestReference.test.ts passes (7 tests)

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 5, 2026

Warning

Rate limit exceeded

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

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 2 minutes and 0 seconds.

⌛ 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.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 69d3c4ec-1c85-4e26-8a4b-37a5231457b6

📥 Commits

Reviewing files that changed from the base of the PR and between 0f789b5 and 3e0c910.

📒 Files selected for processing (2)
  • apps/web/src/pullRequestReference.test.ts
  • apps/web/src/pullRequestReference.ts
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch merge/upstream-1749-canonicalize-pr-refs

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.

@github-actions github-actions bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS 0-9 effective changed lines (test files excluded in mixed PRs). labels Apr 5, 2026
@aaditagrawal aaditagrawal merged commit 343ffcf into main Apr 5, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 effective changed lines (test files excluded in mixed PRs). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants