Aller au contenu principal

Mob review

Principe

Pour homogénéiser les pratiques, éliminer les remarques récurrentes et chronophages des revues de code, et s'approprier les concepts, on met en place la revue de code à plusieurs a.k.a "Mob Review".

Fonctionnement

Un vendredi sur 2, la mob review passe par deux étapes : "Analyse" et "Mise en commun".

Analyse

  • 2 personnes par MR
  • 1h pour analyser et commenter la MR.
  • Si on n'a pas le temps de tout faire, ce n'est pas grave, l'idée, c'est de mettre en commun.
  • Ne pas hésiter à noter des demandes d'approfondissements sur du code incompris ou des notions inconnues.

Mise en commun

  • On désigne un secrétaire qui met à jour le CR
  • Vingt minutes par revue.
  • La personne qui a fait la MR. Un des reviewers parcourt les commentaires (on sélectionne les plus pertinents). Les reviewers expliquent. On échange.
  • On met à jour la doc si nécessaire
  • Si un sujet important émerge, on planifie une réunion

Si tous les points ne sont pas abordés, pas grave, les MR reprennent le workflow classique : on prend soin de désigner un reviewer qui s'assurera que la revue va à son terme.