Successfully reported this slideshow.
We use your LinkedIn profile and activity data to personalize ads and to show you more relevant ads. You can change your ad preferences anytime.

Une revue de la revue de code

603 views

Published on

Revue de code ou pas? Gerrit ou pull requests? Quelle taille de commit? Dois-je revoir tous les commits?

Quelques éléments de réponse dans cette présentation faite pour Agile Grenoble 2016

Published in: Software
  • Be the first to comment

Une revue de la revue de code

  1. 1. sales@enalean.com - www.enalean.com Manuel Vacelet @vaceletm www.enalean.com Une revue de le revue de code
  2. 2. @vaceletm - www.enalean.com Revue de code ?
  3. 3. @vaceletm - www.enalean.com Stratégies
  4. 4. @vaceletm - www.enalean.com Post commit ● “Le mal est fait” ● Liste de diffusion + patch ● Notification + GUI
  5. 5. @vaceletm - www.enalean.com Pré commit ● “Avant qu’il ne soit trop tard”
  6. 6. @vaceletm - www.enalean.com Pull requests
  7. 7. @vaceletm - www.enalean.com Dashboard
  8. 8. @vaceletm - www.enalean.com Une revue
  9. 9. @vaceletm - www.enalean.com Fichiers & commentaires
  10. 10. @vaceletm - www.enalean.com Workflow master feature1 Pull request ? Contributeur Reviewer
  11. 11. @vaceletm - www.enalean.com Workflow master feature1 :(
  12. 12. @vaceletm - www.enalean.com Bilan ● Avantages – Simple à mettre en place – Apporte rapidement les bénéfices de la revue – Mode le plus populaire Github, Gitlab, Bitbucket, Tuleap, ... ● Inconvénients – Nécessite beaucoup de discipline – Passe difficilement à l’échelle
  13. 13. @vaceletm - www.enalean.com Gerrit
  14. 14. @vaceletm - www.enalean.com Dashboard
  15. 15. @vaceletm - www.enalean.com Une revue
  16. 16. @vaceletm - www.enalean.com Fichiers & commentaires
  17. 17. @vaceletm - www.enalean.com Workflow
  18. 18. @vaceletm - www.enalean.com Workflow master feature1 Contributeur Reviewer Local development Gerrit server git commit --amend patchset +2 submit -1
  19. 19. @vaceletm - www.enalean.com Travailler les commits
  20. 20. @vaceletm - www.enalean.com Bilan ● Avantages – Petit commits – Orienté flux de travail – Git fu ● Inconvénients – Git fu – Difficile pour les nouveaux – Nécessite un “Champion”
  21. 21. @vaceletm - www.enalean.com Le choix d’Enalean
  22. 22. @vaceletm - www.enalean.com Revue: clefs du succès
  23. 23. @vaceletm - www.enalean.com Critères partagés ● Automatiser tout ce qui est possible – Tests automatique – Code style (intégration avec SonarQube) ● Qu’est ce qu’on revoit – Qu’est ce qui doit être corrigé – Qu’est ce que l’on accepte de laisser passer
  24. 24. @vaceletm - www.enalean.com Workflow Take a task from a story Write code push for review Go on gerrit dashboard Is there code I can review ? Is there commit reviewed ? Review code test modification Review code test modification
  25. 25. @vaceletm - www.enalean.com Dashboard & priority queues
  26. 26. @vaceletm - www.enalean.com A retenir ● Coder sans review c’est comme conduire sans ceinture ● Une fois adoptée, cela devient LE sujet récurrent de retro ● Envie de démarrer mais quelle stratégie ?

×