Code review, ce truc qui ne sert à rien.

A presentation at GDG DevFest Nantes in October 2017 in Nantes, France by Sébastien Charrier

Slide 1

#CODE Ce truc qui ne sert à rien. DEVFEST NANTES - 20/10/2017 REVIEW

Slide 2

Sébastien   Charrier scharrier

Slide 3

La code review, QUOI C’EST ?

Slide 4

Slide 5

SVN, OU CVS. À L’ÉPOQUE, NOUS AVIONS

Slide 6

VITE CODER ALLAIT

Slide 7

Slide 8

GIT, Et puis, on a eu GITHUB,
GITLAB, BITBUCKET…

Slide 9

À PLUSIEURS. DE VRAIS OUTILS POUR PRODUIRE

Slide 10

PULL* REQUEST

ou merge

Slide 11

CODE REVIEW On peut enfin pratiquer la à priori.

Slide 12

Faire relire son code par un pair , avant intégration dans le produit.

Slide 13

Mais, ça prend du temps . 99% des boss

Slide 14

Mais, ça prend du temps . 99% des boss OUI

Slide 15

30% AU MOINS DU TEMPS* *estimation personnelle

Slide 16

Slide 17

COÛTE OUI, ÇA

Slide 18

QUALITÉ,

Slide 19

COHÉRENCE,

Slide 20

APPRENTISSAGE.

Slide 21

ON Y GAGNE. Sur le long terme,

Slide 22

Il y a deux éléments importants.

Slide 23

UN ÉTAT
D’ESPRIT

Slide 24

Slide 25

Nous sommes ici pour créer le meilleur produit

possible.

Slide 26

DES GUIDELINES

Slide 27

Slide 28

Elles contiennent les règles de style du code et les règles des reviews .

Slide 29

Pour éviter de se répéter , et pour limiter les conflits .

Slide 30

QUELQUES
RÈGLES

Slide 31

POUR TOUT LE MONDE

Slide 32

JE FAIS PREUVE D’EMPATHIE

Slide 33

JE RESTE HUMBLE JE FAIS PREUVE D’EMPATHIE

Slide 34

JE RESTE HUMBLE JE NE SUIS PAS IRONIQUE JE FAIS PREUVE D’EMPATHIE

Slide 35

JE RESTE HUMBLE JE NE SUIS PAS IRONIQUE JE N’INSULTE PAS* JE FAIS PREUVE D’EMPATHIE

true story.

Slide 36

Slide 37

JE SUIS REVIEWER

Slide 38

JE NE SUIS PAS DIRECTIF

Slide 39

JE NE SUIS PAS DIRECTIF JE SUIS EXPLICITE

Slide 40

JE NE SUIS PAS DIRECTIF JE SUIS EXPLICITE JE MONTRE LE POSITIF

Slide 41

JE NE SUIS PAS DIRECTIF JE SUIS EXPLICITE JE MONTRE LE POSITIF JE SUIS RESPONSABLE

Slide 42

Slide 43

JE SUIS REVIEWÉ

Slide 44

JE LIMITE LA TAILLE DES PRS

Slide 45

JE SUIS RECONNAISSANT JE LIMITE LA TAILLE DES PRS

Slide 46

JE SUIS RECONNAISSANT JE NE PRENDS PAS POUR MOI JE LIMITE LA TAILLE DES PRS

Slide 47

JE SUIS RECONNAISSANT JE NE PRENDS PAS POUR MOI J’EXPLIQUE MES CHOIX JE LIMITE LA TAILLE DES PRS

Slide 48

JE SUIS RECONNAISSANT JE NE PRENDS PAS POUR MOI J’EXPLIQUE MES CHOIX JE RÉPONDS À TOUT JE LIMITE LA TAILLE DES PRS

Slide 49

JE SUIS RECONNAISSANT JE NE PRENDS PAS POUR MOI J’EXPLIQUE MES CHOIX JE RÉPONDS À TOUT JE N’ABANDONNE PAS JE LIMITE LA TAILLE DES PRS

Slide 50

VOUS N’ÊTES PAS D’ACCORD ?

Slide 51

Slide 52

DISCUTEZ DE VISU,

Slide 53

OU DEMANDEZ À
UN TIERS.

Slide 54

Slide 55

DÉCIDE In fine, celui qui maintient Quitte à refaire une PR après.

Slide 56

MERCI. http://bump.sh

Ps s s s t !