SHAPE YOUR COMMITS SHAPE YOUR COMMITS WITH GERRIT WITH GERRIT - PowerPoint PPT Presentation

Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... SHAPE YOUR COMMITS SHAPE YOUR COMMITS WITH GERRIT WITH GERRIT original version online This is a PDF export, Manuel Vacelet @vaceletm Martin Goyot @Erwyn / & / Creative Commons Attribution-ShareAlike 3.0 Unported License This work is licensed under a . 1 of 18 07/08/2014 06:04 PM

BIO / CONTEXT BIO / CONTEXT Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... Tuleap Manuel: Enalean CTO & developer Tuleap Martin: developer Tuleap is a forge 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? Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... Well known practice in FLOSS world Various strategy: patches (LKML), "pull requests", ... 4 of 18 07/08/2014 06:04 PM

GERRIT GERRIT Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... http://code.google.com/p/gerrit/ Comes from Android project Git only Users: Eclipse, Openstack, Sony, Motorola, Google, SAP, ... 5 of 18 07/08/2014 06:04 PM

GERRIT WORKFLOW GERRIT WORKFLOW Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... 6 of 18 07/08/2014 06:04 PM

REVIEWS DASHBOARD REVIEWS DASHBOARD Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... 7 of 18 07/08/2014 06:04 PM

ONE REVIEW ONE REVIEW Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... 8 of 18 07/08/2014 06:04 PM

PATCH PATCH Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... 9 of 18 07/08/2014 06:04 PM

DIFF, COMMENTS DIFF, COMMENTS Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... 10 of 18 07/08/2014 06:04 PM

PUSH FOR REVIEW PUSH FOR REVIEW Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... git push ssh://vaceletm@gerrit.tuleap.net:29418 HEAD:refs/for/master One commit == One review Shape your commits (history rewriting!) 11 of 18 07/08/2014 06:04 PM

COMMITS: SIZE MATTERS COMMITS: SIZE MATTERS Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... Beware The Big One Difficult to grasp it Tiny not better whatsoever 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 PART THE TRICKY PART Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... 14 of 18 07/08/2014 06:04 PM

TULEAP HELP TULEAP HELP Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... Ease of startup project initialization default permissions Large scale deployments permissions template group management 15 of 18 07/08/2014 06:04 PM

CONCLUSION CONCLUSION Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... Pro tip: reduce the wait list Pro tip: damn fast Continuous Integration @EnaleanTeam : highest dev ROI this year 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

CREDITS CREDITS Shape your commits with Gerrit file:///home/manuel/workspace/workspace/presentations/RMLL14_Gerrit... It's all about feedback - code review as a great tool in the agile toolbox Scaling Up JGit Deploying Gerrit Code Review 18 of 18 07/08/2014 06:04 PM