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.

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
9. "corriger bug N+1"
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
58. Bonne Revue De Code! Merci!
59. Questions ?