Revue de code avec l'IA

Dans la dernière leçon, on a appris à écrire une description de PR qui résume plusieurs commits. Maintenant, une fois la PR ouverte et le CI passé, arrive l'étape la plus facile à sauter : lire le diff lui-même, exactement comme on lirait la PR d'un développeur humain. Un CI vert signifie que le cod

Vérifier un diff écrit par un agent, c'est comme inspecter un colis livré chez toi avant de signer le reçu — il ne suffit pas que l'emballage soit fermé et ait l'air correct de l'extérieur, il faut l'ouvrir et confirmer que ce qu'il y a dedans, c'est vraiment ce que tu as commandé, pas quelque chose en plus.

diff de Pull Request: La vue complète de toutes les lignes ajoutées et supprimées dans une PR, au format unified diff ; c'est ce qu'on lit ligne par ligne pour vérifier qu'un changement fait exactement ce qui a été demandé.
demande de changements (Request changes): Un verdict de revue qui bloque la fusion jusqu'à ce que l'auteur corrige des points spécifiques ; différent d'un commentaire libre — il empêche concrètement le merge tant que le point n'est pas traité.
CI vert (et ce qu'il ne signifie pas): Une confirmation que le code a passé le lint, la vérification de types et les tests existants ; ça ne confirme pas que le diff est limité à ce que le ticket demandait, et ça ne vérifie pas les valeurs limites qu'aucun test ne couvre.
changement sans rapport intégré dans un diff: Une modification qui apparaît dans la même PR mais qui n'a aucun rapport avec la correction demandée ; même si elle passe le CI en silence, il faut l'extraire ou la relire séparément.