사용자 대상 기능을 추가, 변경 또는 삭제하거나 Bazel에 중요한 아키텍처 변경을 적용하려는 경우 변경사항을 제출하기 전에 설계 문서를 작성하고 검토를 받아야 합니다.
다음은 중요한 변경사항의 몇 가지 예입니다.
- 네이티브 빌드 규칙 추가 또는 삭제
- 네이티브 규칙의 중대한 변경사항
- 하나 이상의 규칙 동작에 영향을 미치는 네이티브 빌드 규칙 시맨틱 변경사항
- Bazel의 규칙 정의 API 변경사항
- Bazel이 다른 시스템에 연결하는 데 사용하는 API 변경사항
- Starlark 언어, 시맨틱 또는 API 변경사항
- Bazel 성능 또는 메모리에 광범위한 영향을 미칠 수 있는 변경사항 용도 (좋은 쪽과 나쁜 쪽)
- 널리 사용되는 내부 API 변경사항
- 플래그 및 명령줄 인터페이스 변경사항
디자인 검토 이유
디자인 문서를 작성할 때 다른 Bazel 개발자와 조율할 수 있습니다. Bazel의 핵심 팀으로부터 조언을 구하세요. 예를 들어 제안서에 BUILD, WORKSPACE 또는 bzl 파일을 수정하려면 Starlark팀을 검토자로 추가하세요. 설계 문서는 제출 전에 검토됩니다. 그 이유는 다음과 같습니다.
- Bazel은 매우 복잡한 시스템입니다. 무해해 보이는 로컬 변경사항으로 인해 전 세계적으로 심각한 결과가 발생할 수 있습니다.
- 팀은 사용자로부터 많은 기능 요청을 받습니다. 이러한 요청은 기술적 타당성뿐만 아니라 요청할 수 있습니다
- Bazel 기능은 핵심 팀 외부의 사용자가 구현하는 경우가 많습니다. 이러한 참여자의 Bazel 전문성 수준은 매우 다양합니다.
- Bazel팀 자체의 전문성 수준도 다양합니다. Bazel의 모든 부분을 완전히 이해하는 팀원은 없습니다.
- Bazel의 변경사항은 이전 버전과의 호환성을 고려하고 중단을 일으키지 않아야 합니다.
Bazel의 설계 검토 정책은 다음과 같은 가능성을 극대화하는 데 도움이 됩니다.
- 모든 기능 요청은 기본 수준의 검토를 받습니다.
- 적절한 사용자가 설계에 대해 검토한 후에야 작동하지 않을 수 있는 구현에 투자할 수 있습니다.
시작하는 데 도움이 필요하면 Bazel 제안서 저장소. 디자인은 진행 중인 작업이므로 구현 세부정보는 시간이 지남에 따라 또는 의견에 따라 변경될 수 있습니다. 게시된 설계 문서에는 설계가 구현될 때 진행 중인 변경사항이 포함되지 않으며 초기 설계만 포함됩니다. 항상 현재 Bazel 기능에 대한 설명을 참조하세요.
참여자 워크플로
참여자는 디자인 문서를 작성하고 pull 요청을 보내고 제안서 검토자를 요청할 수 있습니다
디자인 문서 작성
모든 디자인 문서에는 다음을 포함하는 헤더가 있어야 합니다.
- 저자
- 마지막 주요 변경 날짜
- 검토자 목록 (한 명만 포함) 수석 검토자
- 현재 상태 (초안, 검토 중, 승인됨, 거부됨, 구현 중, 구현 중)
- 토론 대화목록 링크 (공지 후 추가 예정)
문서는 전 세계에서 읽을 수 있는 Google 문서로 작성하거나 마크다운을 사용하여 작성할 수 있습니다. 아래에서 마크다운 / Google Docs 비교.
사용자에게 표시되는 영향을 미치는 제안에는 이전 버전과의 호환성에 미치는 영향을 설명하는 섹션과 필요한 경우 출시 계획이 포함되어야 합니다.
풀 요청 만들기
설계 문서를 설계 색인에 추가하기 위한 풀 리퀘스트(PR)를 만들어 설계 문서를 공유합니다. 추가 마크다운 파일 또는 PR에 대한 문서 링크를 제공하세요.
가능한 경우 리드 검토자를 선택합니다. 다른 검토자를 참조에 포함합니다. 리드 검토자를 선택하지 않으면 Bazel이 담당자가 PR에 하나를 할당합니다.
PR을 만든 후 검토자는 살펴보겠습니다 예를 들어 리드 검토자가 추가 검토자를 제안할 수 있습니다. 누락된 정보를 찾을 수 있습니다. 리드 검토자는 PR을 승인합니다. 검토 절차가 시작될 수 있다고 생각합니다 제안서가 완벽하다는 의미는 아닙니다. 또는 승인될 예정임 제안서에 제안서가 승인되기에 충분한 정보가 포함되어 있고 논의를 시작할 수 있습니다.
새로운 제안 발표
PR이 제출되면 bazel-dev에 공지사항을 보냅니다.
다른 그룹 (예: bazel-discuss 을 사용하세요.
검토자와 반복 작업
관심 있는 모든 사용자가 제안에 댓글을 달 수 있습니다. 질문에 답하고 제안을 명확히 설명하며 우려사항을 해결합니다.
공지사항 대화목록에서 논의해야 합니다. 제안서가 Google Docs에 있는 경우 댓글을 대신 사용할 수 있습니다(익명 댓글이 허용됨).
상태 업데이트
반복이 완료되면 새 PR을 만들어 제안서 상태를 업데이트합니다. 동일한 리드 검토자에게 PR을 전송하고 다른 검토자를 참조에 포함합니다.
제안을 공식적으로 수락하려면 책임 검토자가 다른 검토자가 결정에 동의하는지 확인한 후 PR을 승인합니다.
첫 공지 후 승인까지 최소 1주일의 간격이 있어야 합니다. 생성할 수 있습니다. 이렇게 하면 사용자가 문서를 읽고 우려사항을 공유할 수 있는 충분한 시간을 갖게 됩니다.
제안이 수락되기 전에 구현을 시작할 수 있습니다. 예를 들어 개념 증명 또는 실험입니다 하지만 변경사항을 제출할 수는 없습니다. 을 검토하세요.
리드 검토자 선택
수석 검토자는 다음과 같은 도메인 전문가여야 합니다.
- 관련 하위 시스템에 대한 지식
- 객관적이고 건설적인 의견을 제공할 수 있음
- 전체 검토 기간 동안 프로세스를 주도할 수 있습니다.
여러 팀의 연락처를 확인해 보세요. 라벨과 함께 사용할 수 있습니다.
마크다운과 Google Docs 비교
둘 다 허용되므로 가장 적합한 방법을 결정하세요.
Google Docs 사용의 이점:
- 시작하기 쉽기 때문에 브레인스토밍에 효과적입니다.
- 공동 편집
- 빠른 반복.
- 수정사항을 간편하게 제안할 수 있습니다.
마크다운 파일 사용의 이점:
- 연결할 수 있는 정리된 URL을 만듭니다.
- 버전에 대한 명시적 기록입니다.
- 링크를 공개하기 전에 액세스 권한을 설정하는 것을 잊지 마세요.
- 검색엔진으로 쉽게 검색할 수 있습니다.
- 미래지향적: 일반 텍스트는 특정 도구에 종속되지 않으며 인터넷 연결이 필요하지 않습니다.
- 작성자가 더 이상 존재하지 않더라도 업데이트할 수 있습니다.
- 이러한 작업은 자동으로 처리될 수 있습니다(비활성 링크 업데이트/감지, 저자 목록 가져오기 등).
먼저 Google 문서에서 반복한 다음 나중에 사용할 수 있도록 Markdown으로 변환할 수 있습니다.
Google Docs 사용하기
일관성을 위해 Bazel 디자인 문서 템플릿을 사용합니다. 필요한 헤더를 포함하고 시각적 개체를 다른 Bazel 관련 문서와 일관성이 있어야 합니다. 이렇게 하려면 파일 > 사본 만들기를 클릭하거나 이 링크를 클릭하여 설계 문서 템플릿의 사본을 만듭니다.
누구나 문서를 읽을 수 있도록 하려면 공유 > 고급 > 변경... '사용 - 링크가 있는 모든 사용자'를 선택합니다. 문서에 댓글을 허용하면 Google 계정이 없어도 누구든지 익명으로 댓글을 달 수 있습니다.
마크다운 사용
문서는 GitHub에 저장되며 GitHub 버전의 마크다운 (사양).
PR을 만들어 기존 문서를 업데이트합니다. 주요 변경사항은 문서 검토자가 검토합니다. 사소한 변경사항(예: 오타, 서식)은 누구나 승인할 수 있습니다.
검토자 워크플로
검토자가 설계 문서를 코멘트하고 검토하고 승인합니다.
일반 검토자의 책임
설계 문서를 검토하고 추가 검토 과정을 통과한 설계를 승인합니다.
새 제안서를 받는 경우
- 문서를 빠르게 살펴보세요.
- 중요한 정보가 누락되었거나 디자인이 맞지 않는 경우 언급 프로젝트 목표에도 부합할 수 있습니다
- 추가 검토자를 제안합니다.
- 검토할 준비가 되면 PR을 승인합니다.
검토 프로세스 중
- 문제가 있는 문제에 대해 디자인 저자와 대화를 나눕니다. 명확한 설명이 필요할 수 있습니다
- 필요하다면 설계를 살펴봤습니다
- 댓글 작성의 전제 조건으로 작성자가 처리해야 할 댓글 결정 승인을 받아야 합니다.
- 제안서의 현재 상태에 만족하면 토론 대화목록에 'LGTM'(Looks Good To Me)이라고 작성합니다.
모든 디자인 검토 요청에 이 절차를 따르세요. 디자인 승인 안 함 Bazel이 영향을 받지 않는 경우 설계 색인.
책임 검토자의 책임
구현에 대해 실행 여부를 결정할 책임은 크리에이터에게 있습니다. 설계할 수 있습니다 이 작업을 수행할 수 없는 경우 (대리인에게 PR을 재할당하거나) 버그를 추가 처리를 위한 Bazel 관리자
검토 절차 중
- 의견 및 설계 반복 프로세스가 건설적으로 진행되도록 합니다.
- 승인에 앞서 다른 검토자의 우려가 문제가 해결되었습니다
모든 검토자가 승인한 후
- 메일링 목록에 공지된 후 1주일이 지났는지 확인합니다.
- PR에서 상태를 업데이트했는지 확인합니다.
- 제안서 작성자가 보낸 PR을 승인합니다.
디자인 거부
- PR 작성자가 PR을 보내야 합니다. 또는 PR을 보내야 합니다.
- PR이 문서의 상태를 업데이트합니다.
- 디자인을 승인할 수 없는 이유를 설명하는 메모를 문서에 추가합니다. 현재 상태를 설명하고, 필요한 경우 다음 단계를 간략하게 설명합니다 (예: '잘못된 다시 제출')을 입력합니다.