코드 리뷰로 성장하는 리뷰이의 자세
(insight.infograb.net)-
하나의 이슈에는 하나의 코드 리뷰를
-코드 맥락을 충분히 이해하지 않아도 리뷰할 수 있게 가능한 작은 단위로 Merge Request를 생성해 리뷰어가 이를 빠르게 확인하고 merge 하도록 함 -
리뷰어가 빠르게 이해할 수 있도록 최대한 많은 맥락 제공
-회사나 팀에서 이미 설정한 이슈 템플릿이 있다면 해당 템플릿 사용
-그렇지 않다면 ‘코드가 어떻게 동작하는지’가 아닌 ‘왜 이렇게 코드를 짰는지’ 작성
-이를 뒷받침하는 공식 문서 링크도 추가
-작업 내용을 중심으로 확인받고 싶은 부분 미리 적기 -
빠른 시일 내에 리뷰 받도록 최선 다하기
-‘코드를 작성하고 리뷰어를 선택하는 것까지’만 내 업무가 아니라, ‘리뷰 받은 내용을 반영해서 다시 업데이트한 다음 최종적으로 승인 받는 것까지(내 기능을 merge 하는 것까지)’가 내 업무
-리뷰어를 선택하기 전에 리뷰어가 휴가 중인지, 너무 바쁘진 않은지, 빠르게 대체할 수 있는 사람이 있는지 알아보기
-‘리뷰를 언제까지 받을 수 있는지’ 확인하기 -
리뷰어의 코멘트에 피드백 남기기
-리뷰어가 코드 리뷰를 마치고 피드백을 남겼다면, 그것을 반영해 코드를 수정한 다음, 다시 코드 리뷰 요청
-리뷰어가 남긴 코멘트에 명확하게 피드백을 남겨서 리뷰어가 확실하게 상황을 이해할 수 있도록 하기 -
코드는 내가 아니라는 점 명심하기
-코드 리뷰를 감정적으로 받아들이지 말기
-코멘트를 많이 받았다면 다음에 동일한 코멘트를 받지 않도록 실력 향상하기