💻 잘못된 코드리뷰를 하고 있지 않나요? 2명 이상의 개발팀이라면, 숙명적으로 코드를 서로 공유하게 됩니다. 서로 어떤 코드를 작성했는지를 알기 위해서도 그렇고, 작업간의 통일성을 위해서도 팀

💻 잘못된 코드리뷰를 하고 있지 않나요? 2명 이상의 개발팀이라면, 숙명적으로 코드를 서로 공유하게 됩니다. 서로 어떤 코드를 작성했는지를 알기 위해서도 그렇고, 작업간의 통일성을 위해서도 팀이 커지면 커질수록 코드리뷰는 필연적이게 됩니다. 오늘 소개해드릴 아티클은 코드리뷰에 대한 아티클입니다. 코드리뷰에 대해 이야기하기 전에 우선, 코드리뷰가 아닌 것들이 있습니다. ✅ 코드리뷰는 디자인 리뷰가 아닙니다. 합의된 설계에 대한 리뷰는 해서는 안됩니다. ✅ 코드리뷰는 테스트가 아닙니다. 코드 변경의 동작에 대해 보느라 시간을 보내면, 귀중한 개발자의 시간을 낭비하고 있는 분명한 신호입니다. 코드 리뷰의 핵심 목표는 "개발 속도를 높이고, 지식 배포를 개선하는 것"입니다. 동시에 개발팀 전체가 같은 지식 도메인 갖게 하는데 있습니다. ✔ 레벨 0 - 완전 자동화된 리뷰 사람이 관여하지않는 코드 형식,코드스타일 위반등을 확인합니다. ✔ 레벨 0.5 - 사람의 판단으로 자동화된 리뷰 너무나 자동화된 프로세스는 오히려 코드품질이 저하될 수 있기 때문에 일부는 개발자의 판단이 들어가면 좋습니다. ✔ 레벨 1 - 순수한 가독성 검토 린한 버전의 코드리뷰를 원하신다면 바로 여기입니다. 간단하게 물어보십쇼 "해당 코드를 읽을 수 있습니까?" ✔ 레벨 1.5 - 언어 가독성 검토 큰 조직에서는 언어 가독성이 중요해집니다. 이는 조직 전반에 걸쳐 마찰을 줄이고, 언어 지식을 높일 수 있습니다. ✔ 레벨 2 - 소유권 검토 소유권 검토의 목적은 유지관리자가 코드의 소유권을 가져가도 되는지에 대한 확인을 하는 것입니다. 대부분의 프로젝트는 코드 리뷰의 레벨은 낮은것부터 구축해야합니다. 불필요한 마찰을 줄일 뿐더러, 본질적인 코드리뷰에 근접하게 됩니다. 저는 아래와 같은 리뷰 방식을 추천합니다. - 소규모 오픈소스 : 레벨0 + 레벨2 - 중형 오픈소스 : 레벨0 + 레벨1 + 레벨2 - 대규모 오픈소스 : 레벨0 + 레벨0.5 + 레벨1 + 레벨2 - 단일 팀 회사 : 레벨0 + (레벨0.5) + 레벨1 - 다팀 회사 : 레벨0 + 레벨0.5 + 레벨1 + 레벨1.5 + 레벨2