코드리뷰 무엇을 위해 해야할까?

마지막 경력에서 코드리뷰를 정말 열심히 했었는데요 😇 그 때의 기준을 문서화한 아티클입니다. 코드리뷰는 우리 코드를 위해서, 팀을 위해서, 나를 위해서도 필요합니다 :) 💘코드리뷰란? Github에서 PR이 생성될 때. 개발자가 코드 difference를 특정 branch에 merge하고 싶을 때. 해당 코드 responsibility를 가지고 있는 개발자들이. 동료가. 해당 repository (혹은 코드) 책임자가. 배포하는 코드의 안정성과, 퀄리티 컨트롤, 그리고 코드 작성한 사람의 프로그래밍 능력치 성장을 위해 - 코드의 안정성 Git에 최종으로 저장되는 코드들은 그 최신버전이 배포에 반영되기 때문에 해당 코드가 버그가 없는지, 배포되었을 때 문제가 생기지 않을지 철저히 확인해야 한다. 또한 해당 PR이 반영하고자 하는 스펙이 정확히 구현되었는지 확인해야 한다. 배포되는 브랜치 (develop, release, master 등)가 아닌 경우 위 중요도가 다소 적지만 모든 브랜치에서 항상 안정적으로 완성된 코드여야 결과적으로 그 브랜치들이 merge 될 배포 브랜치의 안정성도 높아진다. - 퀄리티 컨트롤 버그 발생 여부, 스펙 구현 여부와는 별도로 함께 작성하는 코드인만큼 개발자들끼리 정한 규칙을 맞춘다. 코드 컨벤션에 맞춘 코드 스타일 통일, 안쓰는 코드나 파일 삭제, 안티패턴 수정 등이 포함된다. - 성장 코드리뷰를 통해 리뷰어도, 코드 작성자도 함께 성장할 수 있다. 더 나은 코드를 제안하거나 이해되지 않는 코드는 물어보고 내 것으로 만들 수 있도록 한다. 특히 리뷰어는 리뷰 코멘트를 추가할 때 리뷰 받는 사람이 성장할 수 있도록 상세히 설명하고, 질문해서 팀이 성장할 수 있도록 기여한다.