끌어오기 요청을 연 후 검토자는 병합하기 전에 코드를 개선하는 데 도움이 되는 피드백을 남겨 줍니다. 해당 피드백을 해결한다는 것은 각 주석을 이해하고, 해당 주석이 요구하는 변경을 수행하고, 모든 것을 해결했는지 확인하는 것을 의미합니다.
검토 피드백 이해하기
검토자는 일반적인 피드백을 남기고, 특정 줄에 주석을 달고, 정확한 변경 내용을 제안으로 제안할 수 있습니다. 검토는 끌어오기 요청 타임라인에 표시되므로 토론을 따르고 응답이 여전히 필요한 주석을 확인할 수 있습니다. 리포지토리 설정에 따라 검토자는 병합하기 전에 해결해야 하는 끌어오기 요청 또는 요청 변경 내용을 승인할 수도 있습니다.
아무것도 변경하기 전에 각 의견을 읽고 의도를 이해합니다. 주석은 버그를 지적하거나, 질문을 하거나, 다른 접근 방식을 요청하거나, 직접 수락할 수 있는 편집을 제안할 수 있습니다.
Copilot에 액세스할 수 있다면 리뷰 댓글을 해석하고 수정 사항을 제안하는 데 도움이 될 수 있으며, 이는 풀 리퀘스트에서 처리해야 할 댓글이 많을 때 유용합니다.
변경 내용 구현 및 코드 수정
피드백을 처리할 때 다음을 수행할 수 있습니다.
- 검토자가 제안한 변경 사항을 직접 적용하면 해당 제안이 사용자의 브랜치에 커밋됩니다.
- 로컬에서 더 폭넓게 수정한 후 새 커밋을 브랜치에 푸시하세요.
대부분의 피드백은 코드를 업데이트하고 끌어오기 요청 분기에 새 커밋을 푸시하여 해결됩니다. 피드백을 처리할 때 사용자와 검토자 모두 수행된 작업과 주의가 필요한 작업을 추적할 수 있도록 대화를 해결된 것으로 표시합니다.
필수 검토자들이 승인했고 요청된 변경 사항이 더 이상 남아 있지 않다면, 풀 리퀘스트는 병합 단계로 진행할 준비가 되었습니다. 풀 리퀘스트는 브랜치를 추적하므로, 새 커밋이 추가될 때마다 풀 리퀘스트가 업데이트되고 자동 검사가 다시 실행됩니다.
피드백을 효율적으로 처리하기
더 크거나 더 자세히 조사된 끌어오기 요청에서 피드백을 신속하게 해결하는 데 도움이 되는 몇 가지 사례가 있습니다.
- 제안을 일괄 수락하면 제안된 변경 사항을 일괄 작업에 추가하여, 제안마다 하나의 커밋이 생성되는 대신 여러 승인된 변경 사항이 하나의 커밋에 반영됩니다.
- 끌어오기 요청 검토가 해결되면 신속하게 병합할 수 있도록 병합 충돌을 사전에 해결합니다.
- 큰 변경 사항을 적용한 후에는 검토 다시 요청을 사용하여 검토자가 풀 리퀘스트를 다시 검토할 준비가 되었음을 알 수 있게 하세요.
- 필요한 검토를 이해합니다. 브랜치에 승인 또는 코드 소유자의 승인이 필요한 경우, "변경 요청" 검토나 승인 취소는 해결될 때까지 병합을 차단할 수 있습니다.
- 끌어오기 요청을 확장하는 대신 주석에 다시 연결되는 문제를 열어 범위를 벗어난 피드백을 추적합니다.
검토를 해결하는 데 도움이 되는 도구
피드백을 혼자서 수작업으로 하나하나 처리할 필요는 없습니다:
-
풀 리퀘스트를 로컬에서 체크아웃하거나 GitHub Codespaces 에서 열어 문제를 재현하고 푸시하기 전에 수정 사항을 테스트할 수 있습니다.
-
검토 중 제기된 보안 이슈를 해결합니다. 변경 내용에 대한 코드 검색 경고가 끌어오기 요청에 표시되므로 병합하기 전에 수정할 수 있습니다.
-
Copilot을 사용하여 피드백을 분석하고, 질문에 답변하고, 해결 방법을 제안하고, 에이전트와 함께 병합 충돌을 해결할 수 있습니다.