Code ReviewBonnes pratiquesCollaboration

Comment Conventional Comments a transformé nos code reviews

Les code reviews sont un moment clé du cycle de développement. Voici comment nous avons adopté une convention simple qui a changé notre façon de collaborer.

Le problème des code reviews sans cadre

Avant d'adopter les Conventional Comments, nos code reviews souffraient d'un problème courant : les commentaires étaient ambigus. Un simple "Ce code pourrait être amélioré" laissait le reviewer et l'auteur dans le flou.

  • Est-ce un bloquant ou juste une suggestion ?
  • Est-ce une opinion ou une vraie demande de changement ?
  • Doit-on en discuter ou corriger directement ?

Ce manque de clarté générait de la friction inutile et ralentissait les merges.

Qu'est-ce que les Conventional Comments ?

Les Conventional Comments sont une convention qui préfixe chaque commentaire de review par un label explicite :

<label> [decorators]: <subject>

[discussion]

Les labels principaux :

  • praise → pour souligner ce qui est bien fait
  • nitpick → remarque mineure, non bloquante
  • suggestion → idée d'amélioration, le reviewer s'en remet à l'auteur
  • issue → problème à corriger (bloquant par défaut)
  • question → curiosité ou demande d'éclaircissement
  • thought → réflexion en cours, non actionnable

Notre adoption chez Indy

Nous avons intégré cette convention progressivement sur 4 semaines.

Semaine 1 : documentation interne

On a ajouté une page dans notre guide de contribution avec des exemples concrets tirés de notre codebase TypeScript/Ruby.

Semaine 2 : adoption volontaire

Quelques membres de l'équipe ont commencé à l'utiliser. Le feedback immédiat a été très positif.

"Je sais maintenant exactement ce que je dois corriger avant de merger." — Dev backend Indy

Semaine 3 : outillage

On a configuré une GitHub Action qui détecte les commentaires de type issue non résolus et bloque le merge si nécessaire.

Semaine 4 : généralisation

L'adoption est devenue naturelle, sans obligation formelle.

Les bénéfices mesurés

Après 3 mois :

  • Temps moyen avant merge des PRs : -18%
  • Nombre de demandes de clarification en message direct : -40%
  • Satisfaction sur le process de review (enquête interne) : +2 points sur 5

Ce qu'on a appris

Les Conventional Comments ne règlent pas tous les problèmes de communication, mais ils créent un contrat implicite entre reviewers et auteurs. Ce contrat réduit l'ambiguïté et rend les reviews plus respectueuses et efficaces.

La prochaine étape : intégrer un linter qui suggère un label quand un commentaire non préfixé est détecté en review.