Successfully reported this slideshow.
We use your LinkedIn profile and activity data to personalize ads and to show you more relevant ads. You can change your ad preferences anytime.
About Gerrit<br />
What is Gerrit?<br />Google이만든 Code ReviewTool<br />Pre-Commit Code Review<br />Web 기반<br />Git전용 툴<br />JAVA기반<br />Apach...
Gerrit history<br />Google은 Mondrian을 개발해서 Code Review시스템으로 사내용으로 사용 함<br />Guido van Rossum이 Mondrian을 변형해서 Rietveld를 Ope...
Gerrit workflow<br />Code Commit<br />Gerrit에 등록<br />Patch 작성<br />Good <br />Change?<br />Need Patch?<br />Reject 하기<br ...
Gerrit rule<br />+1, -1 점은 Verified 포인트<br />+2,-2 점은 Code Review 포인트<br />+2 포인트를 받아야 만 Summit 가능<br />-1,-2 포인트를 받으면 Sum...
Committer Scenario<br />Local Repo에서 코드를 수정한다<br />이때 for-gerrit branch를 만들면 편하다<br />Gerrit으로 Commit을 Push한다<br />Review를...
Tester Scenario<br />E-mail로 신규 Review가 올라온 것을 확인<br />Commit을 Cherry-pick으로 자신의 Local Repo로 Merge 한다.<br />버그가 고쳐졌는지, 기능이...
Reviewer Scenario<br />E-mail로 신규 Review가 올라온 것을 확인<br />Tester가 +1를 줬다면, Review를 시작한다<br />코드에 문제가 있다면 -2를 준다<br />코드에 큰 ...
Demo<br />간단한 시연!<br />
Good or bad?<br />Code Review는 하는게 좋다!<br />소규모 팀이 쓰기엔 과할 수도?<br />한글이 안 된다!<br />해보니깐 처음엔 힘들어도, 좋았음!<br />
Thank you<br />
Upcoming SlideShare
Loading in …5
×

About gerrit

4,738 views

Published on

Published in: Health & Medicine
  • Be the first to comment

About gerrit

  1. 1. About Gerrit<br />
  2. 2. What is Gerrit?<br />Google이만든 Code ReviewTool<br />Pre-Commit Code Review<br />Web 기반<br />Git전용 툴<br />JAVA기반<br />Apache License 2.0<br />
  3. 3. Gerrit history<br />Google은 Mondrian을 개발해서 Code Review시스템으로 사내용으로 사용 함<br />Guido van Rossum이 Mondrian을 변형해서 Rietveld를 OpenSource로 개발함<br />Android Project가 Git를 VCS으로 채택하고 그에 맞게 Rietveld를 수정해서 Gerrit이 탄생함<br />
  4. 4. Gerrit workflow<br />Code Commit<br />Gerrit에 등록<br />Patch 작성<br />Good <br />Change?<br />Need Patch?<br />Reject 하기<br />Commit Abandon<br />Git에 Merge<br />
  5. 5. Gerrit rule<br />+1, -1 점은 Verified 포인트<br />+2,-2 점은 Code Review 포인트<br />+2 포인트를 받아야 만 Summit 가능<br />-1,-2 포인트를 받으면 Summit 불가능<br />사용자는 1개 이상의 그룹에 속함<br />+1 권한과 +2권한이 분리됨<br />Reviewer는 Committer가 지정 가능<br />특정 그룹은 모든 Commit에 Review가능함<br />
  6. 6. Committer Scenario<br />Local Repo에서 코드를 수정한다<br />이때 for-gerrit branch를 만들면 편하다<br />Gerrit으로 Commit을 Push한다<br />Review를 기다린다.<br />+2를 받았다면 Summit을 눌러서 Git에 Merge한다.<br />-1, -2를받았다면 코드를 수정한 후에 다시 Gerrit으로 Push한다.<br />자동으로 PatchSet이 등록이 됨<br />
  7. 7. Tester Scenario<br />E-mail로 신규 Review가 올라온 것을 확인<br />Commit을 Cherry-pick으로 자신의 Local Repo로 Merge 한다.<br />버그가 고쳐졌는지, 기능이 잘 도는지 Test한다.<br />문제가 있다면 -1을 주고, 이유를 적는다<br />잘 동작 한다면 +1을 준다<br />
  8. 8. Reviewer Scenario<br />E-mail로 신규 Review가 올라온 것을 확인<br />Tester가 +1를 줬다면, Review를 시작한다<br />코드에 문제가 있다면 -2를 준다<br />코드에 큰 문제가 없다면 +2를 준다<br />
  9. 9. Demo<br />간단한 시연!<br />
  10. 10. Good or bad?<br />Code Review는 하는게 좋다!<br />소규모 팀이 쓰기엔 과할 수도?<br />한글이 안 된다!<br />해보니깐 처음엔 힘들어도, 좋았음!<br />
  11. 11. Thank you<br />

×