큰 PR의 문제
- 리뷰어가 다 볼 엄두를 못 내 미룹니다
- 대충 보고 승인해 문제가 통과됩니다
- 충돌이 나면 해결이 복잡합니다
- 되돌려야 할 때 범위가 큽니다
이 파트에서 다루는 내용
작게 나누기올리기 전 셀프 리뷰PR 설명 쓰기리뷰 받아들이기
01
작은 PR이 빨리 리뷰되고 안전합니다
첫 작업을 며칠 동안 붙들고 있다가 수십 개 파일을 한 번에 올리면, 리뷰어가 제대로 볼 수 없습니다. 큰 PR은 리뷰가 미뤄지고, 대충 승인되고, 문제가 숨습니다.
작게 나눠 자주 올리는 편이 낫습니다. 리뷰어가 부담 없이 보고, 피드백을 빨리 받아 방향을 잡을 수 있습니다.
작은 PR의 이점
- 리뷰가 빠르고 꼼꼼해집니다
- 방향이 틀렸을 때 일찍 잡힙니다
- 문제가 생겨도 범위가 작습니다
- 이력이 깔끔해 나중에 추적하기 좋습니다
02
올리기 전에 스스로 먼저 봅니다
리뷰를 요청하기 전에 자신의 변경을 diff로 처음부터 끝까지 봅니다. 이 셀프 리뷰만으로도 상당수의 문제를 미리 잡습니다.
Git 코스의 저장소 위생과 이어집니다. 디버깅용 코드, 주석 처리한 코드, 실수로 들어간 파일, 비밀정보가 없는지 이 단계에서 확인합니다.
무엇을 확인하나
- 디버깅용 출력과 임시 코드 제거
- 주석 처리한 죽은 코드 제거
- 의도치 않게 들어간 파일 확인
- 비밀정보나 접속 정보 노출 여부
- 요청 범위 밖 변경이 섞이지 않았는지
diff로 확인
커밋 전 diff를 스스로 보는 습관입니다. 무엇이 올라가는지 모른 채 전체를 추가하지 않습니다. 이 한 번의 확인이 리뷰어의 시간을 아낍니다.
스스로 설명해 본다
각 변경을 왜 했는지 스스로 설명해 봅니다. 설명이 안 되는 코드가 있으면 아직 이해하지 못한 것입니다. 그대로 올리지 않습니다.
03
PR 설명은 리뷰어를 위해 씁니다
코드만 올리고 설명이 없으면 리뷰어가 맥락을 스스로 파악해야 합니다. 무엇을, 왜, 어떻게 검증했는지 적으면 리뷰가 훨씬 수월해집니다.
이슈와 연결
관련 이슈 번호를 링크하면 왜 이 작업을 하는지 맥락이 이어집니다. 나중에 이 코드를 보는 사람도 배경을 찾을 수 있습니다.
스크린샷·로그
화면 변경이면 전후 스크린샷을, 동작 변경이면 확인한 로그를 첨부합니다. 리뷰어가 직접 실행하지 않아도 결과를 볼 수 있습니다.
PR 설명 형식text
## 무엇을
주문 취소 시 재고를 원복하는 로직을 추가했습니다.
## 왜
결제 취소 후에도 재고가 줄어든 채로 남는 버그(#142)를 고칩니다.
## 어떻게
- OrderService.cancel 에서 재고 원복을 호출합니다.
- 이미 취소된 주문은 다시 원복하지 않도록 상태를 확인합니다.
## 검증
- 단위 테스트 추가: 취소 시 재고 원복, 중복 취소 방지
- 로컬에서 결제→취소 흐름으로 재고 수치 확인
## 리뷰 요청 포인트
- 상태 확인 위치가 적절한지 봐 주세요.
- 동시 취소 요청에 대한 처리는 이번 범위에서 제외했습니다.리뷰 요청 포인트를 적으면 리뷰어가 어디에 집중할지 알 수 있습니다. 범위에서 뺀 것을 명시하면 불필요한 지적도 줄어듭니다.
04
리뷰 피드백은 코드에 대한 것이지 나에 대한 것이 아닙니다
첫 리뷰에서 지적을 많이 받으면 위축되기 쉽습니다. 하지만 리뷰는 코드를 더 낫게 만드는 과정이지 사람을 평가하는 자리가 아닙니다. 경력자도 리뷰에서 지적을 받습니다.
지적을 성장의 재료로 받아들이는 사람이 빨리 큽니다. 방어적으로 반응하면 배울 기회를 놓칩니다.
지적과 나를 분리
이 코드는 이렇게 하면 더 낫다는 말은, 당신이 부족하다는 뜻이 아닙니다. 코드와 나를 분리해서 들으면 편하게 배울 수 있습니다.
동의 안 되면 질문으로
무조건 따르는 것도, 무시하는 것도 아닙니다. 이해가 안 되거나 다른 생각이 있으면 왜 그렇게 하는 게 나은지 질문합니다. 리뷰는 대화입니다.
같은 지적 반복 안 되게
한 번 받은 지적은 다음부터 스스로 걸러 냅니다. 반복되는 지적을 메모해 두면 자신의 약한 부분이 보입니다.
감정적 반응은 피함
지적에 방어적으로 대응하면 리뷰어도 말을 아끼게 되고, 결국 손해는 본인입니다. 이해되지 않으면 감정이 아니라 질문으로 풉니다.
정리
작게 나눠 올리고, 스스로 먼저 검토하고, 리뷰어를 위한 설명을 쓰고, 피드백을 재료로 받아들입니다. 첫 PR을 이렇게 시작한 사람은 리뷰를 두려워하지 않고, 리뷰가 성장의 가장 빠른 통로가 됩니다.
체크