코드리뷰는 어떻게 진행되나요?

안녕하세요! 회사마다 다르겠지만 저희는 사내 코딩 컨벤션을 정리한 문서가 있고 일반적인 코드 리뷰 프로세스는 이렇게 진행하고 있어요. 1. 각 팀 (도메인) 별로 브랜치가 존재함 2. 팀의 작업자가 브랜치를 새로 생성해서 작업을 함 3. 작업이 완료되면 팀의 메인 브랜치에 Pull Request를 올리면서 Reviewer를 등록함 4. 모든 Reviewer의 리뷰가 끝나야 팀 메인 브랜치에 해당 작업 브랜치를 머지 코드 리뷰는 회사나 팀마다 스타일이 다 다르고 현재 개발 문화와 인원에 맞게 프로세스를 잡아가는게 중요한 것 같습니다. 물론, 기존 업무 흐름과 생산성에 크게 영향을 주지 않는선에서 프로세스를 잡는건 어려운 것 같습니다 :) 참고해보시면 좋은 글들 첨부합니다! - https://careerly.co.kr/comments/79688 - https://techblog.woowahan.com/7152/

코드 리뷰는 개발자들이 서로의 코드를 검토하고 피드백을 주고받는 과정으로, 코드 품질 향상과 지식 공유, 팀워크 강화에 도움이 됩니다. 각 회사마다 코드 리뷰 방식이 다를 수 있지만, 일반적으로 다음과 같은 절차를 거칩니다. 1. 코드 리뷰 요청: 코드 작성자가 리뷰 요청을 합니다. Git과 같은 버전 관리 시스템에서 브랜치를 생성하고 작업한 후, Pull Request (PR)를 생성하여 리뷰어들에게 리뷰를 요청합니다. 2. 코드 리뷰어 지정: 리뷰어들을 지정합니다. 보통은 팀 내에서 코드에 익숙한 개발자나 관련 전문가를 지정하며, 필요에 따라 여러 명의 리뷰어를 지정할 수 있습니다. 3. 코드 검토: 리뷰어들이 PR의 변경 사항을 검토하며, 코드의 가독성, 로직의 올바름, 성능, 에러 처리, 테스트 코드 등을 확인합니다. 4. 피드백 제공: 리뷰어들은 코드에 대한 피드백을 작성자에게 제공합니다. 이 때, 질문, 제안, 개선 사항 등을 구체적으로 기술하며, 가능하면 왜 그렇게 생각하는지 이유를 함께 설명합니다. 5. 토론 및 수정: 코드 작성자와 리뷰어들이 피드백에 대해 토론하며, 필요한 수정 사항을 논의합니다. 이 과정에서 서로의 의견을 존중하고, 건설적인 대화를 나눕니다. 6. 코드 수정 및 재리뷰: 작성자가 피드백을 바탕으로 코드를 수정하고, 필요한 경우 리뷰어들에게 재리뷰를 요청합니다. 이 과정은 피드백이 충분히 반영될 때까지 반복됩니다. 7. PR 승인 및 병합: 리뷰어들이 수정된 코드에 만족하면, PR을 승인하고 작성자 또는 리뷰어가 코드를 병합합니다.