Aller au contenu principal
Procidatec
Tous les articles
Méthode

Code review asynchrone : nos règles internes

Pas de meeting, pas de blocage. Voici comment on fait fonctionner la code review en équipe distribuée.

5 min de lecture·par Karim Belkacem, Tech Lead Mobile, Procidatec

Règle 1 — PR de moins de 400 lignes

Au-delà, on découpe. La qualité des reviews chute de manière brutale avec la taille.

Règle 2 — Première review sous 4h ouvrées

Sinon le contributeur est bloqué. C'est notre SLA interne, on le respecte.

Règle 3 — Pas de "LGTM" creux

Une review utile pointe au moins une amélioration ou pose une question. Le LGTM seul ne valide rien.

Règle 4 — Le contributeur tranche

Sur un désaccord, le contributeur a le dernier mot, sauf veto sécurité ou architecture. Ça évite les guerres de chapelles.

Vous voulez en discuter ?

Cet article a soulevé une question pour votre boîte ? Échangeons 30 minutes, sans engagement.