La revue de code est omniprésente dans le développement web. Ses objectifs sont multiples : qualité du code, formation des développeurs juniors, diffusion des bonnes pratiques de l'équipe, etc.

Après 6 années passées à commenter le code des autres et à lire leurs commentaires au sein de la même entreprise, il est temps de faire un bilan et de voir si la revue de code a tenu ses promesses. De nombreux articles et présentations expliquent comment implémenter les revues de code de façon efficace, notamment en étant bienveillant envers les autres et en mettant de côté son ego : le fameux « vous n’êtes pas votre code ».

Néanmoins, cette approche ne tient pas toujours compte des dynamiques complexes qui existent au sein d'une équipe, et elle peut même avoir tendance à les effacer. Par exemple, l'horizontalité qui découle de l'idée qu'on parle du code et pas des personnes peut avoir tendance à ignorer (et donc renforcer) les structures de domination sociale existantes. Si les principes de la revue de code bienveillante restent indispensables, il me semble qu'ils ne sont pas suffisants pour garantir une revue de code qui atteigne ses objectifs.

Je partirai d'observations faites au sein de l'équipe de La Ruche qui dit Oui!, de certaines conséquences négatives parfois contre-intuitives de la revue de code (même bienveillante), voire complètement contraires au but recherché : du code parfois difficile à maintenir, certains projets qui n'avancent pas, des développeurs juniors qui n'ont pas toujours la sensation de progresser, et dans le pire des cas du mal-être au travail. En mettant en lumière certains mécanismes insoupçonnés ou ignorés, nous verrons comment la revue de code peut avoir ces effets indésirables, et des pistes de réflexion pour les éviter.

Comments

Please login to leave a comment

Alex Rock at 10:56 on 22 Oct 2020

Très intéressant, je trouve que ça manquait un peu de détails sur le lien entre l'attitude des personnes et les responsabilités de la politique d'entreprise, là c'était plus généraliste, mais merci beaucoup pour ces détails très utiles pour la revue de code !

Jordan Aubert at 11:03 on 22 Oct 2020

Merci pour ce talk qui nous pousse à réfléchir ! :)

Conférence très intéressante, et qui aborde de manière assez générale un sujet que l'on connait tous et toutes. J'aurai apprécié d'avantage de propositions de solutions concrêtes face aux problématiques exposées. La séance de question/réponse à la fin y a en grande partie répondu.

Yann Eugoné at 13:03 on 22 Oct 2020

Sujet délicat s'il en est, difficile à traiter dans son ensemble dans une conférence de ce format.
J'ai apprécié l'axe de développement, peut-être faudrait-il moins s'étendre sur les redites d'autres conférences (qui sont citées) et approfondir plus le point de vue personnel et le rapport effort/bénéfice.