• 리뷰가 필요 없는 MR → 팀장만 확인
• BE MR → 팀장, BE 팀장, 해당 기능이 사용되는 화면 FE 담당자
• FE MR → 팀장, FE 팀장, 해당 화면에서 사용하는 API 담당자

코드 리뷰 룰

코드 리뷰 in 뱅크샐러드 개발 문화 | 뱅크샐러드

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

• “You” 대신 “We” 쓰기.
• 형용사 / 부사 사용하지 않기.
• 대문자로만 이야기하지 않기.
• 빠르게 리뷰하기.
• nit 줄이기.
  • nit: 사소하고 작은 문제. 스타일 가이드 준수, 가독성 개선 등. 전체적으로는 중요하지 않으나 품질 향상에 도움되는 부분.
• 변경 사이즈 줄이기.
• 하나의 PR에는 하나의 관심사만.

[Github] Pull Request를 통해 코드리뷰(Code Review)하는 법