Oui, la revue de code, c’est bien :D
J’ai eu la chance de faire un stage de 6 mois chez Logilab. (Une boite bien cool ou on fait du libre et du python !) Là-bas, le principe de revue était assez poussé :
N’importe qui peut ouvrir un ticket en disant ce qui faut faire (amélioration, bug, etc). Puis, quelqu’un qui bosse sur le projet prend le ticket. Il fait un patch et le soumet à la revue. Ensuite, la forge de Logilab choisit un relecteur au hasard dans une liste. Ce relecteur valide ou non le patch. Si le patch est refusé, le developpeur initial le rebosse en prenant en compte les commentaires du relecteur. Puis le renvoit. Quand le relecteur accepte le patch, il est envoyé à un “intégrateur”. C’est celui/celle qui connait très bien le programme en question et qui juge la pertinance du patch face au ticket. Si refus, retour à la case départ, sinon, le patch est intégré au projet.
Tout ceci peut sembler “superflu”, mais ça garantie la qualité du logiciel sortant.