코드 리뷰의 목적
코드 리뷰는 버그를 잡고, 코드 품질을 높이며, 지식을 공유하는 과정입니다. 40년간 다양한 팀에서 일한 저로서는 효과적인 코드 리뷰가 팀 역량을 크게 향상시킨다고 확신합니다. 코드 리뷰는 비판이 아닌 협업입니다. 목표는 더 나은 코드를 함께 만드는 것입니다.
리뷰어로서의 자세
작성자가 아닌 코드에 집중하세요. '이 코드는'으로 시작하고 '너는'을 피하세요. 질문 형식으로 의견을 제시합니다. '이렇게 하면 어떨까요?', '이 부분이 이해가 안 되는데 설명해주실 수 있나요?' 칭찬도 함께 합니다. 잘 작성된 부분을 언급하세요. 너무 많은 변경을 요구하지 마세요. 핵심적인 것에 집중합니다.
무엇을 리뷰할 것인가
기능이 요구사항을 충족하는지 확인합니다. 로직의 정확성과 엣지 케이스를 검토합니다. 코드 가독성과 유지보수성을 평가합니다. 성능 문제, 보안 취약점을 주의합니다. 테스트가 충분한지 확인합니다. 문서와 주석이 적절한지 봅니다. 코드 스타일은 린터에 맡기고 중요한 것에 집중하세요.
PR 작성자로서의 자세
PR을 작게 유지하세요. 200-400줄이 적당합니다. 대규모 변경은 여러 PR로 나눕니다. PR 설명에 변경 이유와 테스트 방법을 명시합니다. 리뷰 피드백을 방어적으로 받지 마세요. 피드백은 코드에 대한 것입니다. 변경이 불필요하면 이유를 설명하고 대화하세요.
효율적인 프로세스
리뷰는 24시간 내에 응답합니다. 지연되면 작업 흐름이 끊깁니다. 자동화(린터, 테스트, CI)로 사소한 것을 자동 체크합니다. 페어 프로그래밍으로 실시간 리뷰를 대체할 수도 있습니다. 정기적인 팀 회고에서 코드 리뷰 프로세스를 개선합니다. 코드 리뷰는 학습의 기회입니다.
댓글
0