1
L’analyse de code au
service de la qualité
22
33 ans
À 11 ans…
ekino. http://www.ekino.com
@s_leclercq
Stéphane
Leclercq
...13 ans déjà
33
Agile
Pair
Outils
Il était une fois un projet
44
Agile
Pair
Outils
Il était une fois un projet
55
Agile
Pair
Outils
Il était une fois un projet
66
Agile
Pair
Outils
Il était une fois un projet
77
Acrobatie
Solitude
Tournevis
La mise en production
approche
88
Acrobatie
Solitude
Tournevis
La mise en production
approche
99
Acrobatie
Solitude
Tournevis
La mise en production
approche
1010
Acrobatie
Solitude
Tournevis
La mise en production
approche
11
Livrer à temps de la qualité
1212
Kanban
Pair
IDE
On s’adapte au contexte
1313
Kanban
Pair
IDE
On s’adapte au contexte
1414
Kanban
Pair
IDE
On s’adapte au contexte
1515
Kanban
Pair
IDE
On s’adapte au contexte
16
17
18
O_O
19
20
2121
Petite équipe
Garant
Phabricator
2222
Petite équipe
Garant
Phabricator
2323
Petite équipe
Pas de hiérarchie
Garant
Phabricator
2424
Petite équipe
Pas de hiérarchie
Garant
Phabricator
2525
2900 commits en 4 mois
Commentés
740
Validés
2160
2626
Les débuts
2727
• Excellente réception générale
Les débuts
2828
• Excellente réception générale
• Bugs sur développements en cours en baisse
• Introduction de regressions en baisse
• Dette technique en baisse
Les débuts
2929
• Excellente réception générale
• Bugs sur développements en cours en baisse
• Introduction de regressions en baisse
• Dette technique en baisse
• Qualité des livraisons en hausse
Les débuts
3030
• Excellente réception générale
• Bugs sur développements en cours en baisse
• Introduction de regressions en baisse
• Dette technique en baisse
• Qualité des livraisons en hausse
• Plus de partage de la connaissance
• Montée en expertise et compétence plus rapide
Les débuts
3131
• Excellente réception générale
• Bugs sur développements en cours en baisse
• Introduction de regressions en baisse
• Dette technique en baisse
• Qualité des livraisons en hausse
• Plus de partage de la connaissance
• Montée en expertise et compétence plus rapide
• Meilleure maitrise du changement
Les débuts
3232
Malgré tout
3333
Malgré tout Chronophage
3434
Malgré tout Chronophage
Débats
3535
Audits rapides
3636
Audits rapides
Analyse statique
3737
Audits rapides
Analyse statique
Messages de commits
Audit : a1b2c
3838
Audits rapides
Analyse statique
Messages de commits
Audit : a1b2c
Limiter la durée
3939
Limiter les débats
4040
Règles de développement
Limiter les débats
4141
Règles de développement
Objectifs de l’audit
Limiter les débats
4242
Pair
Outils
Équipes plus grandes
4343
Sous équipes (3 – 5)
Pair
Outils
Équipes plus grandes
4444
Sous équipes (3 – 5)
Pair
Outils
Équipes plus grandes
Garant de sous équipe
Garant global
4545
Sous équipes (3 – 5)
Pair
Outils
Équipes plus grandes
Clanisme
Garant de sous équipe
Garant global
4646
Agile
Pair
Outils
Conclusion
4747
Agile
Pair
Outils
Conclusion
4848
Agile
Pair
Outils
Conclusion
4949
Agile
Pair
Outils
Conclusion
5050
Agile
Pair
Outils
Conclusion
5151
@s_leclercq
Questions ?
5252
2 : https://commons.wikimedia.org/wiki/File:Duke_Wave.png
2 : https://www.flickr.com/photos/blakespot/9736880609
4,5,6,47,48,49,50 :
https://secure.flickr.com/photos/heliomedeiros/4512110770
5,6,14,15,48,49,50 : https://secure.flickr.com/photos/esti/4638056301
6,49,50 : https://secure.flickr.com/photos/chris_carter_/6332387890
8,9,10 : https://secure.flickr.com/photos/glowform/3779533414
9,10 : https://secure.flickr.com/photos/torek/14421164021
10 : https://secure.flickr.com/photos/86639298@N02/8559725421
11 : https://secure.flickr.com/photos/71508688@N00/4767540545
13,14,15 : https://secure.flickr.com/photos/orcmid/3879260297
15,50 :
https://commons.wikimedia.org/wiki/File:IntelliJ_IDEA_12.0.4_Ultimate.jp
g
16,17 : https://secure.flickr.com/photos/croytaque/4956387743/
18 : https://secure.flickr.com/photos/philliecasablanca/2455765649
21, 22, 23, 24 : © Facebook
22, 23, 24 : https://secure.flickr.com/photos/infusionsoft/4752463128
24 : https://secure.flickr.com/photos/alainbachellier/6155667900
33, 34, 35, 36, 37, 38 :
https://www.flickr.com/photos/monkeyc/322654818/
36, 37, 38 : https://www.iconfinder.com/icons/10441/gear_icon
37, 38, 43, 44, 45 :
https://www.iconfinder.com/icons/3474/idea_light_bulb_tip_icon
40, 41 : https://www.iconfinder.com/icons/88543/document_write_icon
41 :
https://www.iconfinder.com/icons/49621/chat_speak_talk_translate_icon
45 :
https://www.iconfinder.com/icons/289357/eye_find_search_see_view_icon
Merci pour
les photos


L'analyse de code au service de la qualité

Notes de l'éditeur

  • #3 Sondage sur qui fait de l’analyse, de la programmation en binome, de l’audit de code
  • #12 Comment livrer à temps sans compromettre la qualité ?
  • #13 Il manque un truc La vitesse et les changements de priorités font faire des erreurs supplémentaires (pas de recul) Il n’y a plus assez de pair programming pour voir tous les soucis L’analyse statique ne détecte pas tout
  • #14 Il manque un truc La vitesse et les changements de priorités font faire des erreurs supplémentaires (pas de recul) Il n’y a plus assez de pair programming pour voir tous les soucis L’analyse statique ne détecte pas tout
  • #15 Il manque un truc La vitesse et les changements de priorités font faire des erreurs supplémentaires (pas de recul) Il n’y a plus assez de pair programming pour voir tous les soucis L’analyse statique ne détecte pas tout
  • #16 Il manque un truc La vitesse et les changements de priorités font faire des erreurs supplémentaires (pas de recul) Il n’y a plus assez de pair programming pour voir tous les soucis L’analyse statique ne détecte pas tout
  • #19 Rien ne remplace les yeux des développeurs ! On a décidé de faire de l’audit de code Permet de voir et valider (ou commenter) les changements commit par commit Bloquant (Gerrit, pull request) ou non bloquant (Phabricator)
  • #20 Rien ne remplace les yeux des développeurs ! On a décidé de faire de l’audit de code Permet de voir et valider (ou commenter) les changements commit par commit Bloquant (Gerrit, pull request) ou non bloquant (Phabricator)
  • #21 Rien ne remplace les yeux des développeurs ! On a décidé de faire de l’audit de code Permet de voir et valider (ou commenter) les changements commit par commit Bloquant (pull request, Gerrit) ou non bloquant (Phabricator)
  • #22 Mise en place immédiate pour avoir un retour d’experience rapide et ajuster Petite équipe (8 personnes) Tout le monde peut auditer tout le monde Un ou deux garants
  • #23 Mise en place immédiate pour avoir un retour d’experience rapide et ajuster Petite équipe (8 personnes) Tout le monde peut auditer tout le monde Un ou deux garants
  • #24 Mise en place immédiate pour avoir un retour d’experience rapide et ajuster Petite équipe (8 personnes) Tout le monde peut auditer tout le monde Un ou deux garants
  • #25 Mise en place immédiate pour avoir un retour d’experience rapide et ajuster Petite équipe (8 personnes) Tout le monde peut auditer tout le monde Un ou deux garants
  • #33 Peut devenir chronophage Gérer la perception de l’outil hors équipe Eviter les conflits et débats sur l’utilité Rendre l’outil moins intrusif
  • #34 Peut devenir chronophage Gérer la perception de l’outil hors équipe Eviter les conflits et débats sur l’utilité Rendre l’outil moins intrusif
  • #35 Peut devenir chronophage Gérer la perception de l’outil hors équipe Eviter les conflits et débats sur l’utilité Rendre l’outil moins intrusif
  • #36 Petite équipe (8 personnes) Tout le monde peut auditer tout le monde Un ou deux garants
  • #37 Petite équipe (8 personnes) Tout le monde peut auditer tout le monde Un ou deux garants
  • #38 Petite équipe (8 personnes) Tout le monde peut auditer tout le monde Un ou deux garants
  • #39 Petite équipe (8 personnes) Tout le monde peut auditer tout le monde Un ou deux garants
  • #40 Nécessité de “règles de développement” Sensibilisation sur les objectifs de l’audit Ordre / Suggestion ou discussion Développeur / Développement
  • #41 Nécessité de “règles de développement” Sensibilisation sur les objectifs de l’audit Ordre / Suggestion ou discussion Développeur / Développement
  • #42 Nécessité de “règles de développement” Sensibilisation sur les objectifs de l’audit Ordre / Suggestion ou discussion Développeur / Développement
  • #43 Attention à la noyade Équipes plus grandes Organisation par sous équipes (5 personnes max) Un garant pour 3 – 4 sous équipes Surveillance du clanisme
  • #44 Attention à la noyade Équipes plus grandes Organisation par sous équipes (5 personnes max) Un garant pour 3 – 4 sous équipes Surveillance du clanisme
  • #45 Attention à la noyade Équipes plus grandes Organisation par sous équipes (5 personnes max) Un garant pour 3 – 4 sous équipes Surveillance du clanisme
  • #46 Attention à la noyade Équipes plus grandes Organisation par sous équipes (5 personnes max) Un garant pour 3 – 4 sous équipes Surveillance du clanisme