Quand s'arrêter et vérifier — l'auto-relecture avant le push
Jusqu'ici, on a appris à vérifier une PR déjà ouverte, une fois le CI déjà passé. Dans cette leçon, on recule d'un cran, au moment juste avant même de pousser (push) : le moment où l'agent a fini de modifier le code, et tu es encore seul(e) face au diff, avant qu'il ne devienne le problème de quelqu
Avant de sortir de la maison avec un sac préparé, on le rouvre et on vérifie soi-même ce qu'il y a vraiment dedans. Il ne suffit pas que la fermeture éclair soit fermée et que ça ait l'air en ordre de l'extérieur — il faut s'assurer que rien ne s'y est glissé sans qu'on l'ait mis exprès.
- auto-relecture (self-review) avant le push
- L'habitude de lire le diff complet et de lancer des vérifications locales avant de pousser du code, pour attraper un problème avant qu'il ne devienne une PR publique que quelqu'un d'autre devra exposer.
- git diff --stat
- Un aperçu rapide de tous les fichiers modifiés et du nombre de lignes changées dans chacun — la première étape de l'auto-relecture, avant de plonger dans les détails de chaque fichier.
- effet de bord silencieux
- Une modification qui apparaît dans le même diff mais qui n'a aucun rapport avec la tâche demandée ; elle passe le CI en silence car aucun test existant ne la vérifie, et seule la lecture du diff lui-même la révèle.
- vérification locale avant push
- Lancer le lint et les tests sur ta propre machine avant de pousser, pour attraper les problèmes au moment le moins coûteux à corriger — avant qu'ils ne deviennent un retour public sur une PR ouverte.
