Add manual mentor PR verification#140
Conversation
|
@saurabhhhcodes is attempting to deploy a commit to the codersogs-3057's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Hey @saurabhhhcodes You have 4 open PRs right now. The limit is 3 at a time. Please get your existing PRs merged or closed before opening new ones:
This PR will remain open but won't be reviewed until you're under the limit. See our Contributing Guidelines for details. |
|
Quick update: one of my MergeShip PRs was closed, so I am back under the open-PR limit now. This PR is ready for review whenever you get a chance; I tested the manual mentor verification flow changes locally and kept it scoped to the issue. |
|
Please see this #67 (comment) |
|
Closing this to respect the repo's open-PR limit and keep maintainer review focused. This branch is now behind/dirty and the maintainer note points to the broader discussion, so I do not want this older PR to block review of the newer, cleaner scoped PRs.\n\nThanks for the feedback, Ayush. I will revisit this flow only after the discussion direction is clear and with a fresh, smaller branch if needed. |
Summary
verifyMentorPrserver action for L2+ manual mentor verificationxp_eventsand trigger the existing mentor comment workflowFixes #88
Testing
git diff --check✅Note
I could not run Vitest/typecheck locally in this Codex shell because this checkout has no
node_modulesand no package manager binary (npm,pnpm,yarn,npx, orcorepack) available on PATH. The implementation keeps the server action as the trust boundary and uses existing XP cap / mentor-comment primitives.GSSoC / NSoC Labels Requested
Please add
gssoc:approved,level:advanced,quality:exceptional,type:feature,type:backend,type:design,nsoc26, andlevel3if this PR meets the program criteria.