Skip to content

Wire up REVIEW_ASSIGNEE_EMAIL for hand-off - #250

Open
jodavis-claude wants to merge 1 commit into
mainfrom
fix/set-review-assignee-email
Open

Wire up REVIEW_ASSIGNEE_EMAIL for hand-off#250
jodavis-claude wants to merge 1 commit into
mainfrom
fix/set-review-assignee-email

Conversation

@jodavis-claude

Copy link
Copy Markdown
Collaborator

Summary

  • final-sign-off and work-with-pr reference $REVIEW_ASSIGNEE_EMAIL to look up the human reviewer's Jira/GitHub identity at hand-off, but it was never set anywhere — every task's PR hand-off stalled on the reviewer-assignment step (hit on both ADR-338 and ADR-339).
  • Sets it to jodasoft@outlook.com (jodavis) per explicit user confirmation.
  • Note: PRs in this repo are authored under a separate jodavis-claude account (confirmed via a live gh api review-post — see PR ADR-338: Wire mypy --strict static analysis into ml/ build #246's review), so the GitHub review-request step should NOT hit a self-review conflict as originally assumed.

Test plan

  • After merge, confirm a task's hand-off successfully assigns the Jira issue and requests jodavis as GitHub reviewer.

🤖 Generated with Claude Code

https://claude.ai/code/session_01NhMgWJeN4ytrcZdaWZYgqY

final-sign-off and work-with-pr reference \$REVIEW_ASSIGNEE_EMAIL to
look up the human reviewer's Jira/GitHub identity at hand-off, but it
was never set anywhere, so every task's PR hand-off stalled on the
reviewer-assignment step (hit on both ADR-338 and ADR-339).

Note: GitHub rejects a review request where reviewer == PR author.
These PRs are currently authored under the same jodavis account this
resolves to, so the GitHub review-request half of final-sign-off's
step 2 is expected to no-op/skip for now (Jira assignment still
applies) until PR authorship moves to a separate identity.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NhMgWJeN4ytrcZdaWZYgqY
@jodavis-claude
jodavis-claude enabled auto-merge (rebase) July 28, 2026 01:37
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