You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Quan es rep un webhook que s'ha fet push a una pull-request que estiguem de revisors hauriem de tornar a fer una validació automàticament.
Pensar què és millor tenir un model on tenim guardat on som revisors per quan revem webhook ja fer-ho o a cada webhook que revem mirar si som revisors... pensa en eficiencia i si es guarda després s'ha de netejar un cop fet merge (també s'hauria de mirar el webhook)
Implementat a #336: els webhooks pull_request.synchronize tornen a encolar una review només si GitHub confirma que un bot configurat continua sent revisor.
He optat per consultar la timeline paginada (amb fast-path a requested_reviewers) en lloc de persistir estat local: evita desincronització i neteja en merge. El lookup falla tancat, i el SHA nou evita coalescing amb una review antiga en execució.
Validació: 683 tests locals passats i CI completa en verd (dashboard, Python 3.11 i Python 3.12).
Activity
giscebot commentedon Nov 2, 2026
Implementat a #336: els webhooks
pull_request.synchronizetornen a encolar una review només si GitHub confirma que un bot configurat continua sent revisor.He optat per consultar la timeline paginada (amb fast-path a
requested_reviewers) en lloc de persistir estat local: evita desincronització i neteja en merge. El lookup falla tancat, i el SHA nou evita coalescing amb una review antiga en execució.Validació: 683 tests locals passats i CI completa en verd (dashboard, Python 3.11 i Python 3.12).