Slide 1

Slide 1 text

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

Slide 2

Slide 2 text

Sébastien 
 Charrier scharrier

Slide 3

Slide 3 text

C’EST ? La code review, QUOI

Slide 4

Slide 4 text

No content

Slide 5

Slide 5 text

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

Slide 6

Slide 6 text

CODER ALLAIT VITE

Slide 7

Slide 7 text

No content

Slide 8

Slide 8 text

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

Slide 9

Slide 9 text

DE VRAIS OUTILS À PLUSIEURS. POUR PRODUIRE

Slide 10

Slide 10 text

PULL* REQUEST * ou merge

Slide 11

Slide 11 text

CODE REVIEW On peut enfin pratiquer la à priori.

Slide 12

Slide 12 text

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

Slide 13

Slide 13 text

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

Slide 14

Slide 14 text

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

Slide 15

Slide 15 text

AU MOINS 30% DU TEMPS* *estimation personnelle

Slide 16

Slide 16 text

No content

Slide 17

Slide 17 text

OUI, ÇA COÛTE

Slide 18

Slide 18 text

QUALITÉ,

Slide 19

Slide 19 text

COHÉRENCE,

Slide 20

Slide 20 text

APPRENTISSAGE.

Slide 21

Slide 21 text

ON Y GAGNE. Sur le long terme,

Slide 22

Slide 22 text

Il y a deux éléments importants.

Slide 23

Slide 23 text

UN ÉTAT D’ESPRIT

Slide 24

Slide 24 text

No content

Slide 25

Slide 25 text

Nous sommes ici pour créer le meilleur produit possible.

Slide 26

Slide 26 text

DES GUIDELINES

Slide 27

Slide 27 text

No content

Slide 28

Slide 28 text

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

Slide 29

Slide 29 text

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

Slide 30

Slide 30 text

QUELQUES RÈGLES

Slide 31

Slide 31 text

POUR TOUT LE MONDE

Slide 32

Slide 32 text

JE FAIS PREUVE D’EMPATHIE

Slide 33

Slide 33 text

JE RESTE HUMBLE JE FAIS PREUVE D’EMPATHIE

Slide 34

Slide 34 text

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

Slide 35

Slide 35 text

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

Slide 36

Slide 36 text

No content

Slide 37

Slide 37 text

JE SUIS REVIEWER

Slide 38

Slide 38 text

JE NE SUIS PAS DIRECTIF

Slide 39

Slide 39 text

JE NE SUIS PAS DIRECTIF JE SUIS EXPLICITE

Slide 40

Slide 40 text

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

Slide 41

Slide 41 text

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

Slide 42

Slide 42 text

No content

Slide 43

Slide 43 text

JE SUIS REVIEWÉ

Slide 44

Slide 44 text

JE LIMITE LA TAILLE DES PRS

Slide 45

Slide 45 text

JE SUIS RECONNAISSANT JE LIMITE LA TAILLE DES PRS

Slide 46

Slide 46 text

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

Slide 47

Slide 47 text

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

Slide 48

Slide 48 text

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

Slide 49 text

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

Slide 50 text

VOUS N’ÊTES PAS D’ACCORD ?

Slide 51

Slide 51 text

No content

Slide 52

Slide 52 text

DISCUTEZ DE VISU,

Slide 53

Slide 53 text

OU DEMANDEZ À UN TIERS.

Slide 54

Slide 54 text

No content

Slide 55

Slide 55 text

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

Slide 56

Slide 56 text

MERCI. http://bump.sh Psssst !