코드 리뷰 시스템 Crucible Programming

Crucible은 Jira로 유명한 Atlassian에서 만든 코드 리뷰 시스템 입니다.
http://www.atlassian.com/software/crucible/

Code Collaborator에 비해 기능은 적지만 가격이 저렴합니다.

간단하게 리뷰를 올리는 프로세스를 보여드리겠습니다.

리뷰를 올리는 방식은 Post-Review가 있고 Pre-Review가 있습니다.
  • Post-Review : 저장소에 Commit 된 Changelist 를 기반으로 리뷰를 진행
  • Pre-Review : 소스를 Commit 하기전에 리뷰를 올려서 진행
먼저 Post-Review 프로세스 입니다.

대쉬보드에서 Review 탭을 클릭하면 "Create New Review"가 보입니다. 클릭.

Browse Changesets를 클릭

본인이 Commit한 Changelist가 보입니다. 적절한 항목을 선택하고 "Edit Details"를 클릭.
이렇게 바로 Changelist가 나오게 하려면 Crucible의 계정과 소스저장소의 계정이 동일해야 합니다.


리뷰어를 지정하고 "Start Review"를 클릭합니다.


여기서 부터는 Reviewer의 시선입니다. 변경된 파일들이 좌측에 트리구조로 보이는데, 하나씩 클릭해서 보면 됩니다.
이것은 없던 소스가 삽입된 경우.

이것은 기존 소스가 변경된 경우 입니다.

라인을 클릭하면 코멘트를 달 수 있습니다.

반드시 고쳐야하는 심각한 오류는 Defect를 체크하고 심각도, 분류 항목을 선택해 줍니다.

"Complete"를 클릭해서 Reviewee에게 보냅니다.

Reviewee는 코멘트를 확인하고 수정할것 수정한 후 "Summarize"를 클릭하여 리뷰를 정리합니다.


"Close Review"를 클릭하면 리뷰가 종료됩니다.

다음은 Pre-Review 입니다. Pre-Review를 받기 위해서는 소스저장소에서 patch를 만들어야 합니다. 퍼포스를 예로 들면 아래와 같이 명령줄에 입력하면 현재 pending list를 patch.txt로 만들어 줍니다.

p4 diff -dcu > patch.txt

Post-Review에서 Browse Changesets 선택했던 화면에서 Pre-Commit 선택하고 미리 만들어둔 patch.txt를 업로드 하면 됩니다.

이후는 Post-Review와 같습니다.

덧글

  • 뇨릉 2011/12/06 09:17 # 답글

    Pre-Commit이라는게 있었군요~ SVN에서 지원하는 건가요??
    개인적으로는 Pre-Commit을 통해서 리뷰하는게 좋겠다고 생각되네요~
    커밋해놓구 또 변경해서 커밋하는거 바람직하지 않았다고 생각했었거든요. ㅎㅎ
    좋은 리뷰 잘 봤습니다 ^^*
  • 자바워크 2011/12/12 02:48 #

    SVN에서도 가능합니다. patch파일을 생성해서 업로드만 하면 됩니다.
  • 자바워크 2012/04/24 11:29 # 답글

    패치파일 만들때 -dcu100 혹은 -dcu500 이렇게 숫자를 넣으면 변경점 앞뒤로 몇 라인을 포함할지 정할 수 있습니다.(기본은 4)
  • ポリス メガネ 2013/08/23 19:02 # 삭제 답글

    This article may mention Twitter. Really?도움말]
  • 2014/09/06 05:18 # 삭제 답글

    좋은 쓰기까지, 나는 자신의 블로그를 정기적으로 방문자 오전, 우수한 운영까지 유지하고, 그것은 긴 시간 동안 정기적 인 방문자가 될 것입니다.
댓글 입력 영역