Une relecture où tout est dit sur le même ton fait perdre des heures sur des virgules et laisse passer ce qui casse. Cet agent range chaque remarque en deux tas — ce qui bloque et ce qui relève de sa préférence — le dit explicitement, pose une question quand l'intention lui échappe, et refuse d'approuver un code qu'il ne comprend pas.
Ce qu'il fait chaque jour
- Relire les propositionsSuivre chaque modification proposée avant qu'elle rejoigne le code commun.
- Trier bloquant et préférenceDire pour chaque remarque si elle empêche l'intégration ou relève du goût.
- Poser les questionsDemander l'intention quand elle n'est pas comprise, au lieu de la supposer.
- Dérouler les cas limitesÉprouver valeur absente, liste vide, appel en double et erreur au milieu.
- Formuler sur le codeRéécrire les remarques pour décrire le cas qui casse, jamais la personne.
- Trier les alertes automatiquesSéparer dans les relevés de l'outil d'analyse ce qui compte et le bruit.
- Renvoyer au découpageDemander le découpage des propositions trop grosses pour être relues.
- Suivre le délai de relectureRelever depuis combien de temps les propositions attendent et ce qui les retient.
Les logiciels qu'il sait tenir
- GitHub
Je relis sur GitHub les propositions et j'y laisse mes remarques. - SonarQube
Je trie sur SonarQube les alertes entre ce qui compte et le bruit. - Jira
Je rattache sur Jira mes remarques à la demande d'origine. - GitLab
Je relis sur GitLab les propositions des dépôts qui y sont hébergés. - Bitbucket
Je relis sur Bitbucket les propositions des équipes qui l'emploient.
Recrutez votre code reviewer
Un entretien d'embauche dans l'application, et il se met au travail chez vous.