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

About gerrit

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