피드로 돌아가기
Dev.toCareer
원문 읽기
코드 리뷰의 기술, 관계를 해치지 않고 품질을 높이는 7단계 전략
The Art of Saying "No" in Code Review
AI 요약
Context
코드 리뷰 시 피드백 전달 방식에 따라 기술적 의도가 왜곡되는 문제 발생. 특히 비영어권 개발자의 경우 톤 조절 실패로 인해 의견이 무시되거나 공격적으로 인식되는 한계 존재.
Technical Solution
- 긍정적 관찰(Level 0)을 통해 신뢰 관계를 구축하고 심리적 안전감 확보
- Nit 표기를 활용해 기능에 영향 없는 단순 스타일 수정 사항을 명시적으로 분리
- Consider, Another option 등의 표현으로 선택권을 부여하는 제안 방식 도입
- 질문 형태의 피드백을 통해 작성자의 체면을 유지하며 스스로 오류를 깨닫게 하는 전략
- 추상적 비판 대신 구체적인 데이터와 컨텍스트를 결합한 액셔너블 피드백 제공
- 문제 정의(What)와 해결 방법(How)을 분리하여 작성자의 설계 자율성 존중
Impact
- O(n²) 루프를 Map 기반 O(n) 구조로 개선 시 50K 데이터 기준 처리 시간 3s에서 5ms로 단축
Key Takeaway
코드 리뷰는 단순한 버그 탐색이 아닌 커뮤니케이션 설계 과정임. 정교한 피드백 레벨 설정을 통해 팀의 심리적 안전감과 코드 품질을 동시에 높이는 엔지니어링 문화 구축 가능
실천 포인트
강제 사항이 아닌 스타일 수정은 반드시 'Nit:' 접두사를 붙여 PR 블로킹 요소를 제거할 것