Shape your commits with Gerrit … - Shape your... · 2014. 7. 8. · SSHHAAPPEE YYOOUURR...

SHAPE YOUR COMMITSSHAPE YOUR COMMITSWITH GERRITWITH GERRIT

This is a PDF export, / & /

This work is licensed under a .Creative Commons Attribution-ShareAlike 3.0 Unported License

@ErwynMartin Goyot@vaceletmManuel Vacelet

original version online

Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

1 of 18 07/08/2014 06:04 PM

BIO / CONTEXTBIO / CONTEXTManuel: Enalean CTO & developerMartin: developerTuleap is a forge

TuleapTuleap

Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

2 of 18 07/08/2014 06:04 PM

WHY TO REVIEW CODE?WHY TO REVIEW CODE?Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

3 of 18 07/08/2014 06:04 PM

WHERE IT COMES FROM?WHERE IT COMES FROM?Well known practice in FLOSS worldVarious strategy: patches (LKML), "pull requests", ...

Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

4 of 18 07/08/2014 06:04 PM

GERRITGERRIT

Comes from Android projectGit onlyUsers: Eclipse, Openstack, Sony, Motorola, Google, SAP, ...

http://code.google.com/p/gerrit/

Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

5 of 18 07/08/2014 06:04 PM

GERRIT WORKFLOWGERRIT WORKFLOWShape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

6 of 18 07/08/2014 06:04 PM

REVIEWS DASHBOARDREVIEWS DASHBOARDShape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

7 of 18 07/08/2014 06:04 PM

ONE REVIEWONE REVIEWShape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

8 of 18 07/08/2014 06:04 PM

PATCHPATCHShape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

9 of 18 07/08/2014 06:04 PM

DIFF, COMMENTSDIFF, COMMENTSShape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

10 of 18 07/08/2014 06:04 PM

PUSH FOR REVIEWPUSH FOR REVIEWOne commit == One reviewShape your commits (history rewriting!)

git push ssh://vaceletm@gerrit.tuleap.net:29418 HEAD:refs/for/master

Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

11 of 18 07/08/2014 06:04 PM

COMMITS: SIZE MATTERSCOMMITS: SIZE MATTERSBeware The Big OneDifficult to grasp itTiny not better whatsoever

Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

12 of 18 07/08/2014 06:04 PM

SHAPE YOUR COMMITS!SHAPE YOUR COMMITS!Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

13 of 18 07/08/2014 06:04 PM

THE TRICKY PARTTHE TRICKY PARTShape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

14 of 18 07/08/2014 06:04 PM

TULEAP HELPTULEAP HELPEase of startup

project initializationdefault permissions

Large scale deploymentspermissions templategroup management

Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

15 of 18 07/08/2014 06:04 PM

CONCLUSIONCONCLUSIONPro tip: reduce the wait listPro tip: damn fast Continuous Integration

: highest dev ROI this year@EnaleanTeam

Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

16 of 18 07/08/2014 06:04 PM

QUESTIONS?QUESTIONS?Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

17 of 18 07/08/2014 06:04 PM

CREDITSCREDITSIt's all about feedback - code review as a great tool in theagile toolboxScaling Up JGitDeploying Gerrit Code Review

Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit...

18 of 18 07/08/2014 06:04 PM