Code review - DevFest Nantes

Vous venez de terminer de coder une nouvelle fonctionnalité pour votre projet. Elle est parfaite, tout est clair, vous l'avez testée dans tous les sens. Normal, le code, ça vous connaît.

Et là, un type se ramène et commence à vous faire de petites remarques. Il vous parle de cohérence, de refactoring, de patterns à appliquer. Il vous fait même remarquer que ligne 42, vous avez oublié un espace avant le point virgule. Et puis il se barre, en vous donnant une petite tape sur l'épaule, et en vous souhaitant bon courage. Vous pleurez.

Si c'est l'image que vous avez de la code review, alors arrêtez tout, et venez. Les code reviews nous aident à garder la cohérence technique d'un produit, sa maintenabilité, sa stabilité, et surtout nous permettent d'apprendre des autres, continuellement. Evidemment, il y a des règles à respecter pour que ça fonctionne. Mais le résultat est là.

Une perte de temps ? Un truc inutile ? On en reparle.

Vidéo: https://www.youtube.com/watch?v=6aQK6GoTbxM

Transcript

1. #CODE Ce truc qui ne sert à rien. DEVFEST NANTES - 20/10/2017 REVIEW
2. Sébastien 
 Charrier scharrier
3. C’EST ? La code review, QUOI
5. À L’ÉPOQUE, NOUS AVIONS SVN, OU CVS.
6. CODER ALLAIT VITE
8. GIT, Et puis, on a eu GITHUB, GITLAB, BITBUCKET…
9. DE VRAIS OUTILS À PLUSIEURS. POUR PRODUIRE
10. PULL* REQUEST * ou merge
11. CODE REVIEW On peut enfin pratiquer la à priori.
12. Faire relire son code par un pair, avant intégration dans le produit.
13. Mais, ça prend du temps. 99% des boss
14. Mais, ça prend du temps. 99% des boss OUI
15. AU MOINS 30% DU TEMPS* *estimation personnelle
17. OUI, ÇA COÛTE
18. QUALITÉ,
19. COHÉRENCE,
20. APPRENTISSAGE.
21. ON Y GAGNE. Sur le long terme,
22. Il y a deux éléments importants.
23. UN ÉTAT D’ESPRIT
25. Nous sommes ici pour créer le meilleur produit possible.
26. DES GUIDELINES
28. Elles contiennent les règles de style du code et les règles des reviews.
29. Pour éviter de se répéter, et pour limiter les conflits.
30. QUELQUES RÈGLES
31. POUR TOUT LE MONDE
32. JE FAIS PREUVE D’EMPATHIE
33. JE RESTE HUMBLE JE FAIS PREUVE D’EMPATHIE
34. JE RESTE HUMBLE JE NE SUIS PAS IRONIQUE JE FAIS PREUVE D’EMPATHIE
35. JE RESTE HUMBLE JE NE SUIS PAS IRONIQUE JE N’INSULTE PAS* * true story. JE FAIS PREUVE D’EMPATHIE
37. JE SUIS REVIEWER
38. JE NE SUIS PAS DIRECTIF
39. JE NE SUIS PAS DIRECTIF JE SUIS EXPLICITE
40. JE NE SUIS PAS DIRECTIF JE SUIS EXPLICITE JE MONTRE LE POSITIF
41. JE NE SUIS PAS DIRECTIF JE SUIS EXPLICITE JE MONTRE LE POSITIF JE SUIS RESPONSABLE
43. JE SUIS REVIEWÉ
44. JE LIMITE LA TAILLE DES PRS
45. JE SUIS RECONNAISSANT JE LIMITE LA TAILLE DES PRS
46. JE SUIS RECONNAISSANT JE NE PRENDS PAS POUR MOI JE LIMITE LA TAILLE DES PRS
47. JE SUIS RECONNAISSANT JE NE PRENDS PAS POUR MOI J’EXPLIQUE MES CHOIX JE LIMITE LA TAILLE DES PRS
48. JE SUIS RECONNAISSANT JE NE PRENDS PAS POUR MOI J’EXPLIQUE MES CHOIX JE RÉPONDS À TOUT JE LIMITE LA TAILLE DES PRS
49. JE SUIS RECONNAISSANT JE NE PRENDS PAS POUR MOI J’EXPLIQUE MES CHOIX JE RÉPONDS À TOUT JE N’ABANDONNE PAS JE LIMITE LA TAILLE DES PRS
50. VOUS N’ÊTES PAS D’ACCORD ?
52. DISCUTEZ DE VISU,
53. OU DEMANDEZ À UN TIERS.
55. DÉCIDE In fine, celui qui maintient Quitte à refaire une PR après.
56. MERCI. http://bump.sh Psssst !