코드 리뷰는 코드 품질을 높이는 중요한 과정이다. 하지만 잘못하면 팀 분위기를 해칠 수 있다. 효과적인 코드 리뷰는 기술과 커뮤니케이션 모두 필요하다.
코드가 아닌 사람을 비판하지 말자. 이 코드는 복잡합니다 대신 이 코드를 더 단순하게 만들 수 있을까요?라고 물어보자. 의문형으로 제안하면 방어적 반응이 줄어든다.
무엇을 볼 것인가
버그를 찾는 것만이 아니다. 가독성, 유지보수성, 테스트 커버리지를 본다. 코드가 의도를 명확하게 표현하는지 확인한다. 미래에 이 코드를 수정할 사람을 생각하자.
아키텍처와 설계도 리뷰한다. 이 변경이 전체 시스템에 어떤 영향을 미치는가? 기존 패턴과 일관성이 있는가? 더 나은 접근 방법은 없는가?
효율적인 리뷰
PR은 작게 유지하자. 500줄이 넘으면 집중력이 떨어진다. 큰 변경은 작은 PR로 나눠서 요청하도록 권장하자. 리뷰 품질이 좋아진다.
자동화할 수 있는 것은 자동화하자. 포맷팅, 린팅, 타입 체크는 CI에서 처리한다. 사람은 기계가 잡지 못하는 것에 집중한다. 로직, 설계, 가독성 같은 것들.
긍정적인 피드백도 중요
좋은 코드를 발견하면 칭찬하자. 이 부분 정말 깔끔하네요라고 말하면 동기 부여가 된다. 비판만 있는 리뷰는 지치게 만든다. 긍정과 부정의 균형을 맞추자.
Categories:
