코드 리뷰 원칙
1. 리뷰를 공식적인 개발 일정으로 포함합니다.
- 리뷰 일정을 고려하지 않아 코드 품질과 타협하는 일이 발생하지 않도록 합니다.
2. 의견보다 사실을 주장합니다.
- 사실에는 출처가 따라야 하고, 의견에는 이유가 따라야 합니다.
3. 주장의 blocker 여부를 분명히 합니다.
- 단순한 대안 제시가 요청자에게는 강한 반대로 인식되어서는 안됩니다.
- 이는 명확히 하는 것은 리뷰어의 몫입니다.
3. 프리뷰가 필요하지 않은지를 먼저 고민합니다.
- 구현(WHAT)보다 중요한 것은 설계(HOW)입니다.
- 설계를 논의해야 하는 시기는 리뷰 단계가 아닌 프리뷰 단계입니다.
4. 리뷰 수단을 리뷰 도구에 한정하지 않습니다.
- 리뷰는 커뮤니케이션이며 댓글은 이를 위한 최고의 수단이 아닙니다.
- 가능한 모든 수단을 사용하여 토론하되 결론은 반드시 기록해야 합니다.
5. 상대방의 시간이 낭비되지 않도록 배려합니다.
- 요청자는 코드의 작성 배경, 의도된 동작, 테스트 방법 등을 충분히 설명해야 합니다.
- 온전한 리뷰가 늦어진다면 부분적인 리뷰를 단계적으로 진행하여 빠르게 피드백합니다.