tuleapcon2017-pull requests for easy code review

22
1 Pull Requests for easy code reviews in Tuleap Luc Jeanniard - Intrapreneur - Lead Projet - Scrum Master - Business Developper

Upload: tuleap

Post on 28-Jan-2018

127 views

Category:

Technology


0 download

TRANSCRIPT

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

Sogilis 2017 - Confidentiel

Excellence techniqueBusiness Model DesignAide au financement

3

Développement critique

Prototypage rapide

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

CloudNative- Informatique

embarquée

Sogilis 2017 - Confidentiel

Les coûts de la non qualité

4

Sogilis 2017 - Confidentiel

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

5

Sogilis 2017 - Confidentiel

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

6

Sogilis 2017 - Confidentiel

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

7

Sogilis 2017 - Confidentiel

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

Sogilis 2017 - Confidentiel

Les ingrédients de la qualité logiciel

9

Sogilis 2017 - Confidentiel

La complexité cyclomatique

10

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

Sogilis 2017 - Confidentiel

La couverture de code par les tests

11

Sogilis 2017 - Confidentiel

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

Sogilis 2017 - Confidentiel

Le nombre de WTF par minute !

13

Team room Team room

Sogilis 2017 - Confidentiel

La connaissance collective

14

Sogilis 2017 - Confidentiel

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

15

Sogilis 2017 - Confidentiel

Métriques automatisées VS Revues de code

16

VS

Sogilis 2017 - Confidentiel

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

Sogilis 2017 - Confidentiel

Les pull requests

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

18

Sogilis 2017 - Confidentiel

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

19

Sogilis 2017 - Confidentiel

Assez parlé !

20

Sogilis 2017 - Confidentiel 21

Sogilis 2017 - Confidentiel