- le reviewer fait des commentaires
- l'auteur effectue des changements puis inscrit le numéro du commit en réponse, ou répond par commentaire si pas de changements nécessaires.
- lorsque l'auteur a répondu à tous les commentaires d'un reviewer, il lui refait une demande de revue
- l'auteur regarde les changements faits. Grâce aux numéros de commit en commentaires, il peut très facilement voir les changements faits, et tout le monde peut s'y retrouver plus facilement.
- si l'auteur trouve encore des choses à redire, on revient à l'étape 1. : le reviewer rajoute un commentaire dans un fil si besoin, ou refait un nouveau fil. Il marque comme "résolues" les fils qui le sont. C'est à lui de les marquer comme tel et pas à l'auteur de la PR, le but étant qu'un fil ne soit "résolu" que lorsque les 2 parties sont d'accord.
- si les changements conviennent au reviewer, il approuve la PR
- l'auteur peut merge lorsque les reviewers ont accepté sa PR
Note : les reviewers peuvent s'inscrire comme "assignees" sur la PR pour signifier qu'ils sont entrain de faire leur revue. Ainsi, si l'auteur veut merge, il attend que tous les reviewers qui ont commencé leur revue la finissent, afin d'éviter de merge une PR alors qu'un reviewer est entrain de faire sa revue.
Note : pour récupérer le numéro du dernier commit de la branche, utiliser l'alias :
alias gloc="git log --pretty=format:'%h' | head -1 | tr -d '\n' | xclip -sel clip"