Faire de la revue de code comme un "pro" - Confoo Montréal 2018

Faire de la revue de code comme un "pro" - Confoo Montréal 2018

La revue de code est un super outil pour attraper des erreurs, disséminer les idées et le savoir, et alimenter la collaboration entre les membres de votre équipe. Voyons ensemble le pourquoi et le comment de la revue de code, autant du côté dévelopeur que du côté business. Comment faire une requête de changement facile à réviser, quoi automatiser, à quels détails être le plus attentif et quels sont les gains pour votre entreprise.

5e6bcf291601ee2e0faf35b30a839cb6?s=128

Marc Aubé

March 08, 2018
Tweet

Transcript

  1. Faire de la revue de code Comme Un "Pro"

  2. Mais... À Quoi Ça Sert ?

  3. Réduit Les Coûts

  4. Stimule La Collaboration · permet de briser les silos ·

    favorise l'ownership du projet · "coaching" de devs plus juniors
  5. Améliore La Communication · diffusion de pratiques émergentes · règle

    de l'autobus · conserve les discussions sur le code · facilite le travail avec employés "remote"
  6. Être Un Réviseur Pro

  7. Donnez Du Contexte

  8. None
  9. "corriger bug N+1"

  10. None
  11. Donnez Des Exemples

  12. "Voici comment récupérer le count"

  13. Dirigez Vers De La Documentation

  14. Critiquez Le Code, Jamais La Personne

  15. Félicitez Les Bons Coups Good job, c'est trop propre comme

    système d'alertes !
  16. Posez Des Questions

  17. Répondez Aux Questions

  18. Tenez Vos Revues à < 200-400 loc

  19. Tenez Vos Revues à < 60-90 min

  20. Prenez Au Moins 5 Min *même pour 1 seule ligne

  21. Préparez Une "Checklist" De Points À Valider

  22. Checklist De Réviseur

  23. Être Un Auteur Pro

  24. Ouvrez Un PR/MR Le Plus Tôt Possible

  25. Limitez La Portée De Vos Changements

  26. Revue de 5 lignes Vs 5000 lignes

  27. Faites Des Commits "Logiques"

  28. Racontez Une Histoire * 2eaf071 - revert fix * 75edd04

    - fix bug * 561d6ff - *derp* * c803f9e - ajouter fichiers
  29. Racontez Une Histoire * 2eaf071 - Stocker les pdfs générés

    dans le bucket 2 * 75edd04 - Stocker les images dans le bucket 1 * 561d6ff - Configuration des 2 buckets S3 * c803f9e - Installation du SDK d'AWS via composer
  30. N'hésitez Pas À Donner Du Contexte

  31. Gardez Votre Historique Propre

  32. Faites Un "Auto Peer-Review"

  33. Faites Rouler Les Tests

  34. Annotez Vos Changements

  35. ! Isolez Les Changements Esthétiques

  36. ! Automatisez Les Changements Esthétiques

  37. Gardez Votre Branche À Jour

  38. Gardez Une "Checklist" De Vos Erreurs Courantes

  39. Checklist D'auteur

  40. Vous N'êtes Pas Votre Code

  41. Être Une Équipe Pro

  42. Établissez Vos Standards

  43. Linting (SCSS/LESS/CSS, JS/ES, PHP, etc)

  44. Formattage Automatique

  45. Créez-vous Des Templates

  46. Variez Les Réviseurs

  47. Réservez Du Pour La Revue De Code

  48. Les Outils De Pros

  49. Php Inspections Pour PhpStorm

  50. TextExpander

  51. Développement Collaboratif

  52. Intégration Continue (CI)

  53. PHP-CS-Fixer

  54. Eslint / Stylelint

  55. SensioLabs Security Checker

  56. Script bash

  57. None
  58. Bonne Revue De Code! Merci!

  59. Questions ?