검토하기 좋은 PR

풀 리퀘스트를 한 가지 목적에 집중하기

한 가지 일을 하는 풀 리퀘스트는 그 일에 대한 발견 사항을 받습니다. 리팩터링, 이름 변경, 파일 이동은 동작 변경과 분리하세요. 섞여 있으면 실제로 바뀐 줄이 묻힙니다.

의도를 설명하고 이슈 연결하기

HeyDeer는 제목과 설명을 읽습니다. 변경의 목적, 가장 확신이 없는 부분, 의도적으로 제외한 것을 적으세요. 작성자에게 쓰기 권한이 있으면 “재시도는 범위 밖입니다. #412에서 추적 중” 같은 트레이드오프 설명은 발견 사항을 거절하는 것과 마찬가지로 결정으로 간주됩니다.

이슈도 연결하세요. “연결된 이슈 읽기”가 켜져 있으면(기본값) HeyDeer는 변경이 충족해야 할 요구 사항을 이슈에서 읽을 수 있습니다. HeyDeer가 읽는 내용을 참고하세요.

생성된 파일 표시하기

생성된 코드를 .gitattributes에서 linguist-generated로 표시하면 HeyDeer가 제외합니다. HeyDeer는 이 속성을 베이스 브랜치에서 읽으므로, 이 변경은 먼저 별도의 풀 리퀘스트로 병합하세요.

작업 중에는 초안으로 열기

자동 검토는 기본적으로 초안을 건너뛰며, 검토 준비 완료(ready for review)로 표시하면 풀 리퀘스트를 검토 대상으로 고려합니다. 초기 피드백을 받으려면 초안에 @heydeer review라고 댓글을 남기세요. 검토 필터를 참고하세요.

수정 사항은 새 커밋으로 푸시하기

후속 검토는 마지막 검토 이후 변경된 부분을 살펴보므로, 수정 사항이 새 커밋으로 들어올 때 가장 잘 작동합니다. 대상 브랜치를 병합하거나 그 위로 리베이스해도 괜찮습니다. 관련 수정 사항을 함께 푸시하면 자동 검토 횟수를 아낄 수 있습니다. 후속 검토를 참고하세요.

거절하는 발견 사항에 답글 달기

“수정 안 함”만 적지 말고 스레드에 이유를 답글로 남기거나, 스레드를 해결하세요. 쓰기 권한이 있는 사람이 결정하면 HeyDeer는 그 문제를 다시 제기하지 않습니다. 노이즈 줄이기를 참고하세요.