Re: Tard

La revue de code sur gitlab, c'est carrément de la merde, surtout quand on a déjà utilisé gerrit, qui lui est vraiment un outil adapté, pas une copie de github (qui est naze aussi) :

• Quand on repousse en ayant amendé les patches, les anciens commentaires sont tous simplement perdus... il n'est pas possible de voir si une personne a pris en compte les remarques précédentes car les remarques sont perdues
• Gitlab n'a tout simplement pas de notion de revue, juste des commentaires, du coup il envoie un mail pour chaque commentaire qu'on fait, ça spamme bien et en plus :
  • pas possible pour le destinataire de savoir si la revue d'une personne est finie ou si d'autres commentaires vont arriver de la même personne
  • pas de mode "brouillon", si on met un commentaire et qu'on se rend compte en lisant quelques blocs plus bas qu'en fait le commentaire n'avait pas lieu d'être, c'est mort, il faut vraiment faire gaffe avant d'écrire le moindre commentaire
• Quand quelqu'un vote +1 ou -1 (avec des emojis...) et qu'un patch est modifié, ou ajouté, la note est conservée, alors qu'il faut que la revue du code soit refaite, la note n'est plus d'actualité
• Comparer 2 versions d'un commit n'est pas pratique du tout
• Accessoirement, pas possible de mettre en surbrillance une portion de texte, un commentaire ne peut concerner qu'une ligne