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

Sébastien 
 Charrier scharrier

La code review, QUOI C’EST ?

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

VITE CODER ALLAIT

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

À PLUSIEURS. DE VRAIS OUTILS POUR PRODUIRE

PULL* REQUEST

  • ou merge

CODE REVIEW On peut enfin pratiquer la à priori.

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

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

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

30% AU MOINS DU TEMPS* *estimation personnelle

COÛTE OUI, ÇA

QUALITÉ,

COHÉRENCE,

APPRENTISSAGE.

ON Y GAGNE. Sur le long terme,

Il y a deux éléments importants.

UN ÉTAT
D’ESPRIT

Nous sommes ici pour créer le meilleur produit

possible.

DES GUIDELINES

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

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

QUELQUES
RÈGLES

POUR TOUT LE MONDE

JE FAIS PREUVE D’EMPATHIE

JE RESTE HUMBLE JE FAIS PREUVE D’EMPATHIE

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

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

  • true story.

JE SUIS REVIEWER

JE NE SUIS PAS DIRECTIF

JE NE SUIS PAS DIRECTIF JE SUIS EXPLICITE

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

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

JE SUIS REVIEWÉ

JE LIMITE LA TAILLE DES PRS

JE SUIS RECONNAISSANT JE LIMITE LA TAILLE DES PRS

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

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

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

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

VOUS N’ÊTES PAS D’ACCORD ?

DISCUTEZ DE VISU,

OU DEMANDEZ À
UN TIERS.

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

MERCI. http://bump.sh

Ps s s s t !