Le produitLa revue, c'est le produit.
À chaque sprint, tu ouvres une pull request. Je la reviewe ligne par ligne. Je commente. Souvent je passe en changes requested. Parfois je ne merge pas du tout.
Six critères, les mêmes à chaque fois.
| Axe | La question posée à chaque PR |
|---|
| Correction | Le résultat est-il juste, et comment tu le sais ? |
| Reproductibilité | Est-ce que ça tourne sur un clone frais, chez quelqu'un d'autre ? |
| Lisibilité | Quelqu'un peut-il reprendre ça dans six mois sans t'appeler ? |
| Robustesse | Que se passe-t-il quand l'entrée est vide, dupliquée ou mal formée ? |
| Communication | Est-ce que tu expliques ta décision, ou est-ce qu'on doit la deviner ? |
| Pertinence métier | À quelle question ça répond, et combien ça vaut ? |
Une pull request techniquement parfaite qui ne répond à aucune question métier n'est pas mergée.
C'est la règle. Et c'est un humain qui l'applique.
Et l'autre sensAu sprint 3, tu écris une revue.
Tu ne fais pas que recevoir une revue. Tu en écris une, sur la pull request de quelqu'un d'autre. Trois problèmes s'y cachent, un seul est grave. Savoir le trouver compte. Savoir ne pas commenter les deux autres compte autant.
Aucune formation n'apprend à faire une revue de code. C'est pourtant ce qui sépare quelqu'un qu'on doit accompagner de quelqu'un sur qui l'équipe s'appuie.
Le périmètre, écrit noir sur blancCe qui est en 1:1
- Le diagnostic d'entrée, 45 minutes en direct.
- Sept PR reviewées ligne par ligne, nominatives, sous 48 h ouvrées.
- La revue croisée du sprint 3 : tu reviews la PR de quelqu'un d'autre, et je commente ta revue.
Sans attendre que tu te poses la questionCe qui ne l'est pas
- Les débriefs sont communs, groupés par sprint.
- Il n'y a pas de call hebdomadaire, pas de coaching récurrent, pas de visio à la demande.
- Pendant un sprint, tu poses tes questions par écrit. Je réponds. Je ne débugge pas à ta place : c'est exactement le sujet.
Le 1:1, c'est la revue, pas des visios. Tu n'achètes pas mon agenda. Tu achètes que quelqu'un se mette sérieusement en face de ta PR, sept fois, et refuse de merger quand il faut refuser.