코드 리뷰어에게 보이는 화면. 최종 결론 버튼을 누르면 아래의 팝업이 뜨고 최종 코멘트와 함께 결론낼 수 있다.

코드 리뷰를 진행하면서 유의해야 할 점이 있다. 리뷰이(reviewee), 리뷰어(reviewer) 모두 ‘더 좋은 코드’와 ‘코드 이해’를 위해 코드 리뷰를 한다는 점을 잊어서는 안 된다.

유의할 점

• 공통 사항
  • 코드 수정을 요청할 때는 근거를 함께 제출한다.
• Reviewee
  • 리뷰어의 피드백에 대해, 공격적으로 받아들이면 안 된다. 다만, 비판적으로 받아들일 필요는 있다. 정말로 좋은 제안인지 근거를 보고 판단하여 코드를 수정할지 결정한다.
• Reviewer
  • 코드 중 이해가 안 되는 부분은 질문한다.
  • 최대한 꼼꼼히 보려고 노력하고, 어딘가 적절하지 않은 부분이 있다면 ‘reject’를 한다. (”다음에는 고쳐주세요.”라는 코멘트와 함께 approve를 하면 코드 리뷰를 하는 의미가 없다.)

예시

• 좋은 질문 예시
• 코드 제안 예시

참고하기 좋은 자료 :

GitHub으로 협업하기: 클론부터 코드 리뷰까지

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

코드 리뷰