Code Review
Testing 🟢 JuniorDefinition
Pratique ou un ou plusieurs pairs examinent le code soumis avant son integration. Ameliore la qualite, partage les connaissances et detecte les bugs humainement.
Analogie
Comme la relecture d'un manuscrit par un editeur : il detecte les fautes et les incoherences que l'auteur ne voit plus.
Exemple de code
// Bonne PR review checklist
// [ ] Le code fait ce que la description promet
// [ ] Pas de code mort ou commente
// [ ] Tests inclus pour les nouveaux cas
// [ ] Pas de secrets hardcodes
// [ ] Performance acceptable
// [ ] Nommage clair et coherentCas d'usage
Pour chaque changement de code afin de maintenir la qualite et partager les connaissances dans l'equipe.
Anti-pattern
Approuver les PR sans les lire ('LGTM') ou bloquer les PR avec des nitpicks stylistiques non-automatisables.