GitHub에서 검토 자와 양수인의 차이점은 무엇입니까?


187

GitHub 블로그에 발표 된 2016 년 12 월 7 일에 추가 된 기능 은 풀 요청에 검토자를 추가 하는 옵션 도입했습니다.

GitHub 검토 옵션

이제 공동 작업자에게 명시 적으로 검토를 요청하여 풀 요청을 검토 할 사람을 쉽게 지정할 수 있습니다.

또한 풀 요청 페이지 사이드 바에서 검토를 기다리는 사람들의 목록과 이미 떠난 사람들의 리뷰 상태를 볼 수 있습니다.

그러나 PR에 대한 검토자를 명시 적으로 설정하는 것은 이미 사람을 지정하여 수행되었습니다 ( 피지 정자 옵션).

두 가지 옵션을 모두 사용할 수있게되었으므로 각 옵션의 역할은 모두 동일한 최종 목표를 공유하므로 어떻게됩니까?


1
"할당 자 기능"이 처음 릴리스 될 때 그것을 소개하는 기사가 있습니까?
babeyh

답변:


135

편집하다:

여러 OSS 관리자와 논의한 후, 검토 자 (reviewers) 는 다음과 같이 정의 된 단어 : 검토 대상 (누군가의 코드)과 "피지 명자"가 정의가 느슨한 것으로 정의됩니다.

"검토 자"의 경우 : 코드를 검토하려는 사람입니다. 해당 영역을 담당하거나 커밋을 병합 할 책임이있는 사람 일 필요는 없습니다. GitHub가 자동 제안하는 것처럼 이전에 해당 코드 덩어리에서 작업 한 사람 일 수 있습니다.

"담당자"의 경우 : 프로젝트 팀 / 유지 업체가 의미하는 바에 따라 엄격한 정의가 없습니다. PR 오프너 또는 해당 영역을 담당하는 사람 (검토가 완료된 후 PR을 수락하거나 닫을 사람) 일 수 있습니다. 프로젝트 관리자가 프로젝트에 가장 적합한 것을 열어 두는 것을 정의하는 것은 GitHub의 책임이 아닙니다.

이전 답변 :

좋아요. 계속해서 내 질문에 대답하겠습니다.

쓰기 권한이있는 사용자의 PR : 양수인은 PR을 연 사람과 동일하며 검토자는 이전 양수인 기능 (검토 코드)을 양수인이 선택한 사람입니다.

쓰기 액세스 권한이없는 사용자 (외부 기고자)의 PR : 쓰기 액세스 권한이있는 사용자는 PR (검토 자)을 검토하기 위해 자신 (또는 다른 쓰기 권한 구성원)을 할당합니다. 양수인이 비어 있습니다.

외부 기고자로부터 미완성 된 PR의 경우 : 쓰기 액세스 구성원은 미완성 된 작업을 수행하여 그녀에게 할당합니다. 그녀는 양수인 인 과제를 완수 할 책임이 있습니다 . PR의 주요 이유는 변경 사항을 검토하는 것이기 때문에 변경 사항을 검토 할 다른 사람을 선택합니다.


24
모든 새로운 팀원에 대해이 답변에 대한 링크를 보내서 담당자 및 검토자를 처리하는 방법을 설명해야합니다. 여기서 뭔가 근본적으로 잘못되었다는 생각으로 연결됩니다. (
Andrey Kuleshov

양수인에게 쓰기 권한이 있어야합니까?
Emre Sülün

이메일 알림 동작에 차이가 있습니까?
jxramos

26

GitHub에서 검토자는 풀 요청을 검토하는 사람입니다. 프로젝트 소유자는 모든 관리자로부터 검토를 요청할 수 있으며, 쓰기 권한이있는 관리자 중 하나가 풀 요청을 검토 한 경우에만 풀 요청을 병합 할 수 있도록 옵션을 설정할 수도 있습니다.

공식 github documentation 에 따르면 , 양수인은 특정 문제를 해결하고 요청을 처리하는 사람입니다. 때로는 검토 자로 혼란스러워합니다. 실제로 풀 요청이 아닌 이슈와 함께 사용되므로 문제가 발생하면이를 해결하도록 누군가를 할당 할 수 있습니다. 풀 요청에서, 양수인은 다른 관리자로부터 의견을 받고 변경 요청을받은 후 풀 요청을 병합하는 담당자를 말합니다.


2
답변 주셔서 감사하지만 질문이 완전히 해결되지는 않는다고 생각합니다. 누군가에게 문제를 할당 할 수 있기 때문에 (그녀는 문제 담당자가 될 것입니다.) PR이 전송 될 때 누군가가 검토 자 (PR 담당자)가 될 것입니다.이 시점에서 여전히 담당자와의 차이점에 대해 명확하지 않습니다. 검토 자.
Cezar Augusto

14

허용 된 답변에 따라. 예. "수취인"은 정의가 느슨하고 팀 요구에 맞게 다르게 사용할 수 있습니다.

8 명의 개발자로 구성된 팀의 대부분의 PR에는 변경 사항을 제안하고 궁극적으로 PR을 승인하는 1 명의 검토자가 있습니다. 검토 단계에서 "양수인"은 PR을 연 사람입니다. 나중에 다른 개발자가 PR을 선택하면 새로운 "수취인"이 추가됩니다. PR이 승인되고 QA 또는 직접 병합 준비가되면 새로운 QA "수취인"이 추가됩니다. 이런 식으로 "수취인"목록이 커집니다.

우리는 "할당 자"를 사용하여 다음과 같은 사람들을 집단적으로 지정합니다.

  1. 풀 요청 작성자
  2. PR 변경 제안 작업을하는 저자 (보통 1과 동일)
  3. 품질 관리 담당자
  4. 합병 책임자 (보통 2 또는 3과 동일)

"할당 자"를 사용하면 향후 PR을 쉽게 찾을 수 있습니다. 내 프로젝트 중 하나에> 3000 PR이 있습니다.

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

또는 author:raya-dumas저자가 만든 모든 항목 (문제, PR)을 찾으려면

검색 프로세스를 쉽게하기위한 다른 유사한 쿼리 "이정표"는 PR 검색을 쉽게하기 위해 사용하는 데 매우 유용합니다.

2017 년 4 분기 Github 스크린 샷


잘 설명했다.
Nitin Gaur

저자가 만든 PR을 찾기 위해 author : my-github-handle을 검색 할 수 있습니다.
Wisienkas

1

GitHub 이전에는 담당자 필드 만 있었고 검토 자 필드는 없었습니다. 당시에는 구분이 없었으므로 담당자 필드는 검토 자 필드로 가장 많이 사용되었습니다.

그러나 프로젝트에 적합한 방법으로 사용하십시오.

당사 사이트를 사용함과 동시에 당사의 쿠키 정책개인정보 보호정책을 읽고 이해하였음을 인정하는 것으로 간주합니다.
Licensed under cc by-sa 3.0 with attribution required.