Une revue de la revue de code

sales@enalean.com - www.enalean.com
Manuel Vacelet
@vaceletm
www.enalean.com
Une revue de le revue de code

@vaceletm - www.enalean.com
Revue de code ?

Stratégies

Post commit
● “Le mal est fait”
● Liste de diffusion + patch
● Notification + GUI

Pré commit
● “Avant qu’il ne soit trop tard”

Pull requests

Dashboard

Une revue

Fichiers & commentaires

Workflow
master
feature1
Pull request
?
Contributeur
Reviewer

Workflow
master
feature1
:(

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

Gerrit

Workflow

Workflow
master
feature1
Contributeur Reviewer
Local development Gerrit server
git commit --amend
patchset
+2
submit
-1

Travailler les commits

Bilan
● Avantages
– Petit commits
– Orienté flux de travail
– Git fu
● Inconvénients
– Git fu
– Difficile pour les nouveaux
– Nécessite un “Champion”

Le choix d’Enalean

Revue: clefs du succès

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

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

Dashboard & priority queues

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 ?

Une revue de la revue de code

Recommandé

Recommandé

Contenu connexe

En vedette

En vedette (15)

Similaire à Une revue de la revue de code

Similaire à Une revue de la revue de code (20)

Une revue de la revue de code