개발 프로세스
GitLab에 기여하기 위한 개발 프로세스에 대한 정보는 이 주제를 참조하세요.
프로세스
필독서:
- 기존 구성 요소 조정 및 새로운 구성 요소 소개 가이드
- 코드를 검토하고 검토를 받기 위한 코드 리뷰 가이드라인
- 데이터베이스 관련 변경 사항 및 복잡한 SQL 쿼리를 검토하고 검토를 받기 위한 데이터베이스 리뷰 가이드라인
- 보안 코딩 가이드라인
- GitLab 프로젝트를 위한 파이프라인
- 필수 정지 방지
보충자료:
- GitLab에 기여하기
- 개발자를 위한 보안 프로세스
- 개발자를 위한 패치 릴리즈 프로세스
- 엔터프라이즈 에디션 기능 구현 가이드라인
- GitLab에 새로운 서비스 구성 요소 추가하기
- 변경 로그를 위한 가이드라인
- 종속성
- Danger bot
- GitLab.com에서 ChatOps 접근 요청하기 (GitLab 팀원만 해당)
개발 가이드라인 리뷰
개발 가이드라인 변경 시 경험이 풍부한 GitLab 팀원에게 리뷰 및 승인을 요청하세요.
예를 들어, 특정 그룹에서 독점적으로 사용하는 새로운 내부 API를 문서화하는 경우, 해당 그룹의 구성원 중 한 사람에게 엔지니어링 리뷰를 요청하세요.
오타와 같은 작은 수정은 최소한 유지 관리자 역할을 가진 사용자라면 누구나 병합할 수 있습니다.
광범위한 변경 사항
일부 변경 사항은 둘 이상의 그룹에 영향을 미칠 수 있습니다. 예를 들어:
- 코드 리뷰 가이드라인 변경 사항.
- 커밋 메시지 가이드라인 변경 사항.
- GitLab 개발의 기능 플래그 가이드라인 변경 사항.
- 기능 플래그 문서 가이드라인 변경 사항.
이 경우 다음 워크플로를 사용하세요:
-
팀원의 동료 리뷰를 요청합니다.
-
해당 분야의 책임자인 엔지니어링 매니저(EM) 또는 스태프 엔지니어에게 리뷰 및 승인을 요청합니다:
EM이나 해당 분야의 스태프 엔지니어가 작성한 MR의 경우 이 단계를 건너뛸 수 있습니다.
여러 그룹에 영향을 미치는 경우 각 영향을 받는 분야의 EM/스태프 엔지니어 수준에서 승인이 필요할 수 있습니다.
-
리뷰가 완료된 후, MR의 작성자/승인자인 EM/스태프 엔지니어와 상담하세요.
여러 분야에 걸쳐 중요한 변경 사항인 경우, 개발 가이드라인의 DRI인 개발 VP에게 최종 리뷰 및 승인을 요청하세요.
모든 유지 관리자는 MR을 병합할 수 있습니다.
기술 문서 리뷰
기술 작가의 리뷰를 원하시면 #docs
Slack 채널에 메시지를 게시하세요.
기술 작가가 내용을 리뷰할 필요는 없으며, MR 작성자를 제외한 모든 Maintainer는 병합할 수 있습니다.
리뷰어 가치
리뷰어 또는 리뷰를 받는 사람으로서 GitLab에서 우리가 노력하는 리뷰어 가치를 숙지하세요.
또한, 모든 문서 내용은 문서 스타일 가이드를 따라야 합니다.
언어별 가이드
Go 가이드
셸 스크립팅 가이드
명확한 서면 커뮤니케이션
이슈 또는 병합 요청에 댓글을 작성하거나 다른 커뮤니케이션 모드를 사용할 때,
“MUST”, “MUST NOT”, “REQUIRED”, “SHALL”, “SHALL NOT”, “SHOULD”, “SHOULD NOT”, “RECOMMENDED”, “MAY”,
및 “OPTIONAL”과 같은 용어를 사용할 때 IETF 표준을 따라야 합니다.
이렇게 하면 서로 다른 문화의 다양한 팀원이 사용되는 용어를 명확히 이해할 수 있습니다.