tuleapcon 2017-easy-code-review

1

Pull Requests for easy code reviews in Tuleap

Luc Jeanniard - Intrapreneur - Lead Projet - Scrum Master - Business Developper

Sogilis 2017 - Confidentiel

Sogilis en 2’ top chrono

2


Excellence techniqueBusiness Model DesignAide au financement

3

Développement critique

Prototypage rapide

- 35 personnes- Innovation- Startups- Grands groupes- Application web

CloudNative- Informatique

embarquée


Les coûts de la non qualité

4


Plus un bug est détecté tard, plus son coût est élevé

5


Moins le code est propre, plus son évolution sera difficile

6


Moins le code est propre, plus le travail d’équipe sera difficile et laborieux

7


Hors du logiciel lui même, une question d’image et la réputation de l’entreprise

8

La qualité est essentielle pour la pérennité d’une entreprise


Les ingrédients de la qualité logiciel

9


La complexité cyclomatique

10

C’est le nombre de chemins que peut prendre l'exécution d’une fonction


La couverture de code par les tests

11


Les bonnes pratiques de développement

12

DRY : Do not repeat yourself

YAGNI : You ain't gonna need it Ne faites pas de provision

KISS : Keep it simple, stupid


Le nombre de WTF par minute !

13

Team room Team room


La connaissance collective

14


Les individus et les interactions plus que les process et les outils

15


Métriques automatisées VS Revues de code

16

VS


Le pouvoir des revues de code

✅ Détection des problèmes d’architecture

✅ Détection de bug

✅ Détection de code difficile à comprendre -> difficile à maintenir

✅ Partage / transmission de la connaissance

→ AMÉLIORATION CONTINUE

17


Les pull requests

Rendre systématique les revues de code via un process établi

18


Tuleap offre cette composante essentielle du développement de qualité

19


Assez parlé !

20

Sogilis 2017 - Confidentiel 21

tuleapcon 2017-easy-code-review

Technology