코드 리뷰 가이드라인

GitLab CE 및 EE의 머지 리퀘스트 코드 리뷰 프로세스, 역할별 책임, 승인 가이드라인, 모범 사례를 설명합니다.

GitLab CE 및 EE의 모든 머지 리퀘스트는 코드가 효과적이고, 이해하기 쉽고, 유지보수 가능하며, 안전한지 확인하기 위해 코드 리뷰를 거쳐야 합니다. 머지 리퀘스트 검토, 승인 및 병합 받기 # 시작하기 전에 기여 승인 기준 을 숙지하세요. 그룹 소속 리뷰어 또는 도메인 전문가 에게 코드 검토 를 받으세요. 작고 단순한 변경 사항의 경우 리뷰어 단계를 건너뛰고 메인테이너 에게 직접 요청할 수 있습니다. 작고 단순한 변경의 예시: 오탈자 수정 또는 소규모 텍스트 변경. 동작을 변경하지 않는 소규모 리팩토링. 한 달 이상 기본 활성화된 기능 플래그 제거. 사용하지 않는 메서드나 클래스 제거. 5줄 미만의 코드 변경이 필요한, 잘 이해된 로직 변경. 그 외의 경우, 메인테이너에게 전달하기 전에 MR이 닿는 각 카테고리 의 리뷰어 승인을 받으세요. 보안 지원이 필요한 경우 @gitlab-com/gl-security/appsec 를 포함하세요. 리뷰어가 승인한 후 메인테이너가 검토하고 병합합니다. 마지막으로 필요한 승인자가 병합합니다. CODEOWNERS 필수 승인의 경우, 일반 승인보다 도메인별 승인을 먼저 구하세요. 메인테이너이기도 한 도메인별 승인자는 두 측면을 모두 검토하고 한 번만 승인해야 합니다. 승인 가이드라인 # 아래 메인테이너 책임 섹션에서 설명하는 바와 같이, 도메인 전문성 을 보유한 메인테이너에게 머지 리퀘스트 승인 및 병합을 받도록 권장합니다. 첫 번째 리뷰어의 선택적 승인은 여기서 다루지 않습니다. 그러나 머지 리퀘스트는 개요 섹션에서 설명한 대로 메인테이너에게 전달하기 전에 리뷰어의 검토를 받아야 합니다. 머지 리퀘스트에 포함된 내용 필요한 승인자 ~backend 변경 사항 1 백엔드 메인테이너. ~database 마이그레이션 또는 비용이 많이 드는 쿼리 변경 사항 2 데이터베이스 메인테이너. 자세한 내용은 데이터베이스 리뷰 가이드라인을 참조하세요. ~workhorse 변경 사항 Workhorse 메인테이너. ~frontend 변경 사항 1 프론트엔드 메인테이너. ~UX 사용자 대면 변경 사항 3 프로덕트 디자이너. 자세한 내용은 디자인 및 사용자 인터페이스 가이드라인을 참조하세요. 새로운 JavaScript 라이브러리 추가 1 - 라이브러리가 번들 크기를 크게 증가시키는 경우 프론트엔드 디자인 시스템 멤버.- 새 라이브러리에서 사용하는 라이선스가 GitLab 사용에 승인되지 않은 경우 법무 부서 멤버. 라이선스 호환성에 대한 자세한 정보는 GitLab 라이선스 및 호환성 문서를 참조하세요. 새로운 의존성 또는 파일 시스템 변경 - Distribution 팀 멤버. 자세한 내용은 Distribution 팀과의 협력 방법을 참조하세요.- RubyGems의 경우 AppSec 리뷰를 요청하세요. ~documentation 또는 ~UI 텍스트 변경 해당 DevOps Stage 그룹 담당 기술 작성자. 개발 가이드라인 변경 검토 프로세스에 따라 해당 승인을 받으세요. 엔드투엔드 및 비엔드투엔드 변경 사항 4 테스트 소프트웨