La pyramide de la revue de code

 (morling.dev)
30 points par xguru 2022-04-13 | 4 commentaires | Partager sur WhatsApp
  • Une illustration récapitule en une seule image les points à examiner en priorité lors d’une revue de code et ceux qui devraient être automatisés
  • 5 niveaux : Code Style, Tests, Documentation, Implementation Semantics, API Semantics
  • Les questions essentielles à poser pour chaque niveau
  • Automatiser les aspects comme le style de code et les tests, plus faciles à modifier par la suite, et concentrer la revue sur la documentation, l’implémentation et l’API

À lire aussi

4 commentaires

 
jong9712 2022-04-13

Merci !
 
xguru 2022-04-13

Comme il s’agit d’une image d’une seule page, je l’ai traduite en coréen.
Pyramide de la revue de code : https://drive.google.com/file/d/…
 
galadbran 2022-04-13

J’ai essayé de le corriger, car il y avait quelques points à améliorer : https://drive.google.com/file/d/…

  • L’API est-elle aussi petite que possible et aussi grande que nécessaire ? (corrigé, car il y avait une faute de frappe)
  • La nouvelle dépendance ajoutée apporte-t-elle une valeur suffisante ? (pull one's weight : faire sa part)
  • Les nouvelles fonctionnalités ont-elles été correctement documentées ? (corrigé, car il y avait une faute de frappe)
 
windily 2022-04-13

Merci.