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