코드 리뷰 (for 전체)

저 문맥(Low Context) 커뮤니케이션을 지향하는 문화
코드 리뷰에 임하는 자세
커뮤니케이션 비용을 줄이기 위한 Pn 룰
- P1 - 꼭 반영해주세요.
  - 오류 발생 가능성을 포함하는 등 중대한 코드 수정이 반드시 필요하다고 판단되는 경우.
  - 리뷰 요청자는 이 요청을 반영하거나, 반영할 수 없는 합리적인 의견을 들어 리뷰어를 설득할 수 있어야 함.
- P2 - 적극적으로 고려해주세요.
  - 리뷰 요청자는 이 요청을 수용하거나, 수용할 수 없는 상황이라면 합리적인 의견을 들어 리뷰어와 토론할 것을 권장.
- P3 - 웬만하면 반영해 주세요.
  - 리뷰 요청자는 이 요청을 수용하거나, 다음에 반영할 계획을 표현할 것을 권장.
- P4 - 반영해도 좋고 넘어가도 좋습니다.
  - 요청자는 아무런 의견을 달지 않고 무시해도 괜찮음. 고민해보는 정도면 충분.
- P5 - 그냥 사소한 의견입니다.
  - 무시해도 괜찮음.
리뷰 우선순위 판단을 돕는 D-n 룰
예시
- https://github.com/rosielsh/my-blog/pull/16
- https://github.com/naver/scavenger/pull/61