"Je dois juste modifier cette petite fonction..."
2 semaines plus tard :
→ 100 fichiers modifiés
→ 3000 lignes de code touchées
→ Une PR impossible à review
→ 2 semaines de travail perdues
J'ai malheureusement vu trop de PR comme ça.
Et le pire ? C'est que ça commence toujours pareil.
Un développeur motivé.
De bonnes intentions.
Une envie d'améliorer les choses.
L'histoire est tellement classique :
On doit corriger un bug simple. Vraiment simple.
Mais on voit un bout de legacy qui traine.
"Tant qu'à faire, je vais nettoyer un peu..."
Et là, ça dérape :
→ "Cette classe dépend de celle-là..."
→ "La fonction utilisée est dépréciée..."
→ "Si j'upgrade cette lib c'est encore mieux..."
→ "Je pourrais revoir le code de ça aussi..."
2 semaines plus tard :
Le monstre est là.
Personne n'ose review la PR.
Le temps est perdu.
La solution ?
4 règles simples que j'applique désormais :
1. Une PR = Une seule responsabilité
2. Planifier les impacts avant de coder
3. Traiter le refactoring comme un vrai projet
4. Préférer 10 petites victoires à 1 gros changement
Le vrai skill d'un dev senior ?
Ce n'est pas de tout faire.
C'est de savoir ce qu'il ne faut pas faire.
Et vous, quelle est la plus grosse PR que vous avez croisée ?