🔖 Git으로 협업 시작하기 #9 – 리뷰는 품질이다 : 리뷰어/작성자 역할과 협업 매너

🔹 17. 리뷰어/작성자 역할 분담 기준과 매너

 

📌 17-1. PR은 "검토받을 준비"가 되었을 때만 생성
👉 PR을 리뷰어에게 보내는 건 검토 요청이 아닌 검토 책임 이관

 

➡️ 미완성/충돌 있는 PR은 혼란만 유발

 

✔️ 리뷰 전에 아래 체크리스트 완료하기 :

 

🔍 git diff로 변경 내역 스스로 검토

📂 누락된 파일 없는지 git status 확인

📝 커밋 메시지 논리적이고 목적별로 구성

📄 PR 설명에 기능 요약, 주의사항, 제외 파일 명시

 

 

📌 17-2. 리뷰어는 '방어적 리뷰'가 아니라 '협력적 리뷰'
👉 단순 오류 찾기보다 개선 제안과 대안 제시가 핵심

 

➡️ “이 부분 문제 있음”보다 “이런 방향은 어떨까요?”가 더 생산적

✔️ 코드 컨벤션, 테스트 범위, 성능 영향까지도 함께 고려

 

📌 17-3. 코드에 대해 말하되, 사람을 평가하지 않기

 

❌ “이거 왜 이렇게 했어요?”
✅ “이 부분은 이런 의도로 작성하신 걸까요?”
→ 의도 파악 → 문제 공유 → 개선 방향 합의

 

📌 17-4. 리뷰는 빠르게, 피드백은 명확하게
👉 리뷰 지연은 개발 흐름 전체를 늦춤

 

➡️ 하루 넘는 리뷰 지연은 공유 또는 알림 필요

✔️ 피드백은 애매한 표현 대신 명확하게

 

예 :

❌ “이거 좀 이상한 것 같아요”

✅ “이 부분은 null 가능성이 있으니 Optional 처리 고려 바랍니다”

 

💡 요약 정리
1️⃣ PR은 완성된 상태에서만 생성
2️⃣ 리뷰는 지적보다 제안 중심
3️⃣ 표현은 코드 기준, 사람이 아닌 문제 중심
4️⃣ 피드백은 명확하고 빠르게