Gitlab을 사용하여 코드 검토를 설정하는 방법은 무엇입니까?

Question

Gitlab을 사용하여 코드 검토를 어떻게 설정합니까? Gitlab 웹 사이트에이 기능이 나열되어 있지만 Gitlab 사용자 설명서에 대한 모든 링크가 가장 잘 이해할 수 있도록 링크를 설정하는 방법에 대한 지침을 찾을 수없는 것 같습니다.

내 검색 중 일부는 '병합 요청'이 방법이라고 지적했지만 일부는 제한적이라고 생각합니다. 발행 된 병합 요청은 한 지점과 다른 지점 사이의 모든 커밋을 보여줍니다. 각 개별 커밋에 대해 생성 된 diffs 볼 수있을 것 같습니다. 예를 들어 내가 검토하고 싶은 파일이 있다고 가정 해 봅시다. 새 파일이지만 dev 브랜치에서 10 개 이상의 커밋을 변경 한 내용을 제출했습니다. 통합에서 해당 dev 브랜치에 대한 병합 요청을 발행하면 파일에 대한 증분 변경 내용을 보여주는 10 개의 커밋이 표시됩니다 ... 전체 내용을 검토하고 싶습니다. 그것은 새로운 것입니다!

여기 제가 잘못된 나무를 짖고 있습니까? GitLab에서 사용할 수있는 실제 코드 검토 도구가 있습니까? 아니면 병합 요청을 진행할 수 있습니까? 그리고 내가 잘못 사용하고 있다면 어떻게합니까? 여기에 적절한 코드 검토를 설정하는 가장 좋은 방법은 무엇입니까?

Answer 1

다른 저장소 또는 현재 저장소의 병합 요청에서 제출 된 코드를 볼 수 있습니다. http://demo.gitlab.com/diaspora/diaspora/commits/master

예는 다음 당신은 http://demo.gitlab.com/diaspora/diaspora/commit/42f47626890218a180870bc3f44ec57625b0779c

communication is code review을 결과

예를 커밋 커밋 파일 변경 (버튼 회신) 또는 전체에 대한 주석을 추가 할 수 있습니다. 그러나 개인적으로 가능한 한 얼굴을 맞대고 의사 소통 할 수있는 한 대의 PC에서 코드 검토를 수행하고 결과를 기록하거나 더 많은 형식이 필요할 때 도구를 사용하는 것이 좋습니다.

커밋이 많은 파일 검토의 경우. http://demo.gitlab.com/diaspora/diaspora/blame/master/README.md을 사용하여 누가 그것을했는지 이해할 수 있습니다. 그러나이 견해에는 의사 소통을하고 의견을 추가 할 수있는 옵션이 없습니다. 이 경우에는 변경 사항을 주석으로 추가하는 것이 좋습니다.

Answer 2

예. 병합 요청은 피어 리뷰가 수행되는 방식입니다.

모든 커밋 (여기에서 언급 한 내용 : http://youtu.be/DyAX8ws5OIc?t=3m2s)의 변경 사항을 보여주는 'diff'탭이 있어야합니다.

비디오에서는 피어 리뷰에 어떻게 사용되는지 잘 설명합니다.

Answer 3

저는 Gitlab에서 코드 마찰을 거의 마찰없이 2 개월 이상 해왔습니다. 개발자가 새로운 커밋을 푸시 할 때마다 전자 메일 알림을 보내도록 rss2email을 설정했습니다. 그런 다음 커밋에 Gitlab의 주석 기능을 사용하여 푸시 된 코드에 대한 주석을 작성합니다.

Gitlab은 파일 자체에 대한 주석을 허용하지 않으며 커밋 (Github과 마찬가지로)에서만 주석을 허용합니다. 이전 커밋에서 놓친 부분을 언급해야하는 상황에 처했을 때마다 비난 도구를 사용하여 주석을 달기 위해 코드 섹션을 도입/변경 한 커밋을 찾습니다.

완벽하지는 않지만 지금까지는 좋았습니다.

Answer 4

참고 : GitLab 6.4, side-by-side diff view을 사용할 수 있습니다. "pull request 5308"을 참조하십시오.

(2013 7월) 가능성은 파일 수준에서,하지만 각 라인에 대한 언급을 아직 없습니다.

팀 구성원이 일반적으로 또는 라인 주석이 특정 라인에 병합 요청에 댓글을 달 수 :
Daniel Sokolowski는 Per line comments are now supported (09/2014)이 in the comments 언급하고있다. 여전히 코드 검토 활동에 도움이 될 수 있습니다

.