Je soupçonne une importante révision de code dans mon équipe. Trop de revues de code sont fusionnées sans aucun commentaire.
Il me semble qu'il n'y a pas de révision de code sans un seul commentaire.
Comment puis-je en tant que chef d'équipe contrôler correctement que mon équipe effectue un processus de révision de code approprié et comment puis-je les aider à maximiser les avantages du processus?
Mise à jour
Je pensais que les gens pourraient vouloir être informés de toute mise à jour. J'ai essayé beaucoup de suggestions qui ont été faites ici. la plupart étaient déjà utilisés. certains ont aidé un peu. Cependant, le problème est resté - certaines personnes ont continuellement reçu un mauvais code lorsque je ne regardais pas.
J'ai trouvé que la surveillance de la révision du code n'est pas aussi utile que de donner à mon équipe des outils pour améliorer leur code au départ.
J'ai donc ajouté une bibliothèque nommée "jscpd" pour détecter les pâtes de copie. La construction a échoué sur les pâtes de copie. Cela a éliminé un problème immédiatement.
nous allons ensuite essayer le codeclimat.
Je fais également une revue manuelle des anciennes revues de code une fois par sprint pendant une demi-journée. Je convertis les todo
s en problèmes / tickets - car j'ai découvert que les gens les écrivaient, mais ils ne sont jamais traités ultérieurement. Je fais également des rencontres avec l'ensemble des équipes pour revoir le code quand c'est approprié.
en général, on a l'impression que nous allons dans la bonne direction.