feat: add PR review metrics — reviews given vs received (#253)#586
feat: add PR review metrics — reviews given vs received (#253)#586mallya-m wants to merge 2 commits into
Conversation
|
@mallya-m is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
|
The implementation for #253 is ready for review. Here's a quick summary of what's been added:
Could you please add the following labels?
Happy to make any changes based on your feedback. Thanks! |
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
Two issues:
-
Wrong file path —
src/types/src/types/css.d.tshas a doublesrc/types/src/typesin the path. This should besrc/types/css.d.ts. -
Missing EOF newlines on both
src/types/src/types/css.d.tsand at least one other file.
- Add fetchReviewMetrics() using GitHub GraphQL contributionsCollection - Extend GET route to return reviews alongside PR stats - Add Reviews Given tab to PRMetrics component - Show total reviews, approval rate, and top reviewed repos
9076ec5 to
1a7440e
Compare
|
@Priyanshu-byte-coder fixed both issues:
Ready for merge! |
Summary
Adds a "Reviews Given" tab to the existing PR Analytics card so users can see their code review contributions alongside their authored PRs.
Closes #253
Type of Change
Changes Made
fetchReviewMetrics()inroute.tsusing GitHub GraphQLpullRequestReviewContributionsPRMetrics.tsxHow to Test
npm run devhttp://localhost:3000Screenshots
Checklist
npm run lintpasses locallynpm run type-check)