Code review - DevFest Nantes

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

0a5864849f2b7f748d5949da877c0a32?s=128

Sébastien Charrier

October 20, 2017
Tweet

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

  4. None
  5. À L’ÉPOQUE, NOUS AVIONS SVN, OU CVS.

  6. CODER ALLAIT VITE

  7. None
  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

  16. None
  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

  24. None
  25. Nous sommes ici pour créer le meilleur produit possible.

  26. DES GUIDELINES

  27. None
  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
  36. None
  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
  42. None
  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 ?

  51. None
  52. DISCUTEZ DE VISU,

  53. OU DEMANDEZ À UN TIERS.

  54. None
  55. DÉCIDE In fine, celui qui maintient Quitte à refaire une

    PR après.
  56. MERCI. http://bump.sh Psssst !