Repository navigation
fix: preserve bot PR authorship over assignment - #332
Conversation
Keep changes-requested reviews writable when a bot-authored pull request is also assigned to the bot, with direct email and webhook regressions. Co-authored-by: Eduard Carreras <ecarreras@gisce.net>
pilipilisbot
left a comment
There was a problem hiding this comment.
Aprovat. El canvi de precedència és coherent amb el contracte d'autorització: la menció explícita continua prioritària, però en una PR creada pel bot l'autoria ja no queda emmascarada per l'assignació. Això permet que una review CHANGES_REQUESTED verificada conservi work_allowed tant per IMAP com per webhook; l'autoria d'una issue continua sense autoritzar feina.
He revisat el flux d'enqueue i la revalidació de l'executor, el diff complet i el context de #299. Els tres tests específics d'aquesta correcció passen localment (3 passed); els checks de dashboard, Python 3.11 i Python 3.12 també són verds. No calen canvis de dades ni migracions per aquesta PR.
|
Post-merge sync: #332 is merged as Tracking follow-up: the #332 checkbox in #299 still says “Review and merge” and remains unchecked despite this merge. It should be marked complete in the next authorized issue-status update; the rollout/observation gates remain separate. |
|
Post-merge sync: PR #332 is merged at |
|
Confirmat contra l’estat actual de GitHub: #332 està fusionada a main amb e5eca3f (head revisat 786ea11), però #299 encara conserva sense marcar el checkbox de #332. Aquest run és review_only, així que no he editat l’issue de seguiment ni cap metadata. La propera actualització autoritzada de #299 hauria de marcar només aquest checkbox; els gates d’auditoria, observació i rollout continuen oberts. |
Summary
Why
Job 9274 received a valid changes-requested review on bot-authored PR #330, but assignment was selected before pull request authorship. That masked the authorship signal and left the job in review-only mode instead of allowing the requested fix.
Validation
Refs #299