Skip to content

Autorevisar pull-requests #335

Closed

Description

@ecarreras

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)

Activity

  1. giscebot commented on Nov 2, 2026

    @giscebot
    Collaborator

    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).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions