셩잇님의 잡동사니

코드 리뷰는 조직의 상황에 따라 실행하는 목적도 방식도 목표도 다르다. 그렇기에 정답이 없다. 그래서 내가 코드리뷰에 관한 글을 시작하기 앞서 내가 처한 상황을 설명하지 않는다면, 읽는 사람도 글을 이해하기 쉽지 않을 것이다. 나와 함께 일하는 조직은 아래와 같았다. * 열 명 이하로 구성된 개발 조직이었다. * 대부분의 개발자가 신입 사원이거나 경력이 2년이 채 되지 않는 주니어 개발자였다. * 간혹 10년~20년 경력의 시니어 개발자가 있었으나 코드 리뷰 경험이 없었다. * 실력 있는 개발자를 채용하기 어려웠다. * Git을 SVN(Subversion)처럼 쓰고 있었다. * 이슈 기반의 개발을 하고 있지 않았다. 코드 리뷰어로서 내 목표는 세 가지였다. * 최소한의 코드 품질 유지 * 개발 코칭을 통..

요약 1. 로직을 이해하기 쉽도록 구현했는지를 위주로 코드리뷰합니다. 2. 코드 작성자는 확신이 없는 부분에 대해서 사전에 설명하는 코멘트를 남겨두면 좋습니다. 3. 코드 리뷰어는 자신의 코멘트가 필수적으로 반영되어야 하는지, 선택적으로 반영하면 좋은 것인지 명시하면 좋습니다. 4. 구글 등 다른 코드리뷰 문화도 참고하시면 좋습니다. 경험상 사람마다 코드리뷰에서 주로 보는 부분이 다르겠지만, 저는 로직을 주로 봅니다. 구현하고자 하는 것에 비해서 복잡하게 되어있지는 않은지, 맥락을 알아야지만 이해할 수 있도록 구현되어 있지는 않은지 점검하는 편이에요. 그래서 사실 코드를 읽다가 이해하기 어렵다 싶으면 그때부터 로직에 개선점이 있는지 주의 깊게 봅니다. 코드 스타일 등 린터나 포매터로 자동화할 수 있는 부..