Relire les propositions d'un agent sans perdre de temps
La relecture est le prix à payer pour travailler en sécurité avec des agents. Avec cette méthode, ce prix reste bas.
Une proposition d'agent arrive. Quatre cents lignes modifiées dans dix-huit fichiers. Le développeur soupire, se fait un café et perd le reste de la matinée.
Ce n'est pas une fatalité. La relecture prend du temps, mais combien dépend surtout de la façon dont vous l'organisez.
Gardez les propositions petites
Une proposition de cinquante lignes se relit en cinq minutes. Une proposition de cinq cents lignes vous prend une heure, et vous y laissez passer davantage. Demandez à l'agent de découper les grandes tâches. Un seul sujet par proposition.
Commencez par les tests
Vérifiez d'abord si les tests passent et si de nouveaux tests ont été ajoutés. Vous en savez déjà beaucoup sur la qualité avant d'avoir lu une seule ligne de code. Si les tests échouent, vous renvoyez la proposition sans lire plus loin.
Lisez l'explication, puis le code
Un bon agent explique ce qu'il a fait et pourquoi. Lisez cela en premier. Si le raisonnement ne tient pas, inutile de lire le code.
Sachez quoi vérifier
Vérifiez surtout :
- que la modification fait ce qui a été demandé, et rien de plus,
- que rien ne change dans des parties qui ne figuraient pas dans la consigne,
- qu'aucune donnée sensible ni aucune clé ne se retrouve dans le code,
- que les noms et le style correspondent au reste du code.
Mesurez-le
Notez pendant quelques semaines combien de temps prend la relecture par type de tâche. Vous verrez vite quelles tâches font réellement gagner du temps et lesquelles non. Les tâches qui coûtent plus de temps de relecture qu'elles n'en rapportent, vous les retirez de la liste.
Chez nous, le temps de relecture fait partie de chaque rapport. Un gain de temps qui ignore la relecture n'est pas un gain de temps honnête. Lisez aussi comment nous mesurons le gain de temps.