Embed Size (px)
Citation preview
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://[email protected]: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
