Méthodes de code review selon les entreprises IT

• Pourquoi faire une code review :

  • Motivation : faire preuve de professionnalisme pour ne pas pénaliser le reviewer
  • Partage de connaissances : le partage bidirectionnel des connaissances améliore le niveau technique de toute l’organisation
  • Détection d’erreurs : augmente les chances de repérer les erreurs en amont

• Google :

  • tout le code
  • conformément à des règles alignées sur la politique interne
  • aussi petit que possible, et fréquemment
  • review en moins de 4 heures (moins d’1 heure pour une petite review, moins de 5 heures pour une grande)

• Microsoft

  • 39 % des personnes font une review au moins une fois par jour

• Banksalad

  • effectue de petits PR fréquemment
  • crée des templates avec les changements, les écrans concernés, des checklists pour les tests, etc., afin que le reviewer comprenne facilement et puisse reviewer rapidement

• Woowa Brothers

  • crée et utilise des templates de MR
  • utilise un bot MR pour recevoir des alertes sur « quel MR arrive »

• Toss Payments

  • ajoute parfois un label « Good Review » pour formuler des retours positifs

• Wemakeprice

  • recommande des reviews détaillées et l’usage d’émoticônes

[kuber]

Microsoft : 39 % des personnes font une revue au moins une fois par jour
...?
Ce n’est pas extrêmement peu ? C’est surprenant.

[superwoou]

Ça semble varier selon les entreprises. Chez nous, tous les ingénieurs font des revues au moins deux fois par jour... haha

[ironlung]

J’ai l’impression que chacun l’interprète différemment. ^^;