개발자들이 서로의 코드를 의도적,체계적으로 검토하는 행위.
다른 사람이 작성한 코드를 검토하면서 궁금한 부분 질문하고, 버그가 있는 부분을 알려주고, 효율성이나 개선 해야할 점을 피드백을 주고 받으면서 코드를 개선하는 과정입니다.
- 리뷰어에게 피해주지 않도록 프로의식 발휘하게 됩니다.
- 양방향 지식 공유로 조직 전체 기술력이 향상하고 다른 사람과의 코드 리뷰를 통해 알지못했던 노하우와 경험을 얻을 수 있습니다.
- 자신은 몰랐던 오류를 피드백을 받으면서 오류를 사전 발견할 가능성이 높아집니다.
100% 코드 리뷰, 모든 코드를 리뷰
모든 코드를 사내 정책에 맞는 규칙에 맞춰 최대한 작게,
자주4시간 이내로 리뷰 (작은 리뷰는 1시간 이내, 큰 리뷰는 5시간 이내)
Microsoft의 전체 직원 수는 14만 명이고, 이중 44%인 6만 명에 이르는 개발자의 75%가 매일 코드 리뷰
작은 PR(Pull Request)을 자주 진행함
리뷰어가 쉽게 이해하고 빠르게 리뷰하도록 변경 사항, 관련 화면,
테스트 위한 체크리스트 등을 템플릿으로 만듭니다.
MR(merge request) 템플릿 만들어 사용함
MR봇 사용해 '어떤 MR이 오는지' 알람이 오도록 합니다.
라이브러리 개발과 서비스 개발 모두에서 코드 리뷰가 의무화되어 있습니다. 서비스 코드는 같은 회사의 프론트엔드 개발자들이 리뷰합니다. 라이브러리를 개발할 때에는 모든 프론트엔드 챕터 구성원들이 함께 코드를 리뷰
PR전에 리뷰 요청을 하고 리뷰어들의 승인 후 배포
자세한 리뷰와 이모티콘 사용 권장함
참고사이트
https://www.samsungsds.com/kr/insights/global_code_review.html
https://news.hada.io/topic?id=11048
흥미롭게 읽었습니다 퍼가요!