이 문서에서는 변경사항을 검토하는 방법을 설명합니다. 유효한 변경사항의 기준은 다음과 같습니다.
- 변경사항이 프로젝트의 명시된 목적에 부합하는가?
- 변경사항이 프로젝트의 기존 아키텍처 내에서 유효한가?
- 이러한 변경사항이 앞으로 문제를 야기할 설계 결함으로 이어지는가?
- 이러한 변경사항이 이 프로젝트와 관련하여 수립된 권장사항을 준수하는가?
- 이러한 변경사항이 설명된 기능을 수행하기 위한 좋은 방법인가?
- 이러한 변경사항이 보안 또는 불안정성 위험으로 이어지거나 기존 앱을 중단시키는가?
변경사항 검토에 관한 자세한 내용은 Gerrit 문서의 변경사항 검토 섹션을 참고하세요.
변경사항 검토자로 할당되면 변경사항 링크가 포함된 이메일을 받게 됩니다. 다음 단계에 따라 변경사항을 검토하세요.
이메일의 변경사항 링크를 클릭합니다. Gerrit의 변경사항 정보 화면이 표시됩니다. 이 화면에서는 변경사항에 관한 설명을 제공하고, 변경사항의 기여자 및 모든 검토자를 식별하며, 변경사항과 연결된 파일 목록을 제공합니다.
변경사항 설명을 읽고 변경사항의 목표를 파악합니다.
파일을 클릭하여 파일의 변경사항을 검토합니다. 파일이 두 개의 창 비교 화면에서 열립니다. 왼쪽에는 기존 파일이, 오른쪽에는 제안된 변경사항이 포함된 파일이 표시됩니다. 파일을 검토하려면 다음 단계를 따르세요.
마우스를 사용하여 의견이나 질문이 있는 코드 부분을 강조표시 (클릭 + 드래그)합니다. '의견을 작성하려면 c를 누르세요'라는 메시지가 표시됩니다.
c 를 누릅니다. 초안 의견 상자가 표시됩니다.
의견 또는 질문을 입력하고 저장 을 클릭합니다.
화면 왼쪽 상단의 변경사항 고유 번호를 클릭하여 변경사항 정보 화면으로 돌아갑니다.
변경사항의 각 파일을 계속 검토합니다.
변경사항의 모든 파일 검토를 마치면 변경사항 정보 화면으로 돌아가서 답장 을 클릭합니다. 제출 요구사항 투표 화면이 표시됩니다.
변경사항에 다음 점수 중 하나를 선택합니다.
- +1 괜찮아 보이지만 다른 사람도 승인해야 합니다.
0 점수 없음
-1 수정할 사항이 있으므로 이대로 제출하지 않는 것이 좋습니다.
-2 이 변경사항은 제출해서는 안 됩니다.
답장 을 클릭하여 의견과 점수를 변경사항 개발자에게 보냅니다.
수정할 문제가 있는 경우 개발자가 문제를 수정하고 수정사항을 검토해 달라고 요청합니다. 그렇지 않으면 변경사항 검토가 완료됩니다.