About gerrit

4,385 views
4,092 views

Published on

Published in: Health & Medicine
0 Comments
7 Likes
Statistics
Notes
  • Be the first to comment

No Downloads
Views
Total views
4,385
On SlideShare
0
From Embeds
0
Number of Embeds
65
Actions
Shares
0
Downloads
64
Comments
0
Likes
7
Embeds 0
No embeds

No notes for slide

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 />

×