골
검토 프로세스의 목표는 다음과 같습니다.
- 기능과 가독성 모두에서 고품질 코드를 보장합니다.
- 버그를 포착합니다. 버그가 발생하기 때문입니다.
- 코드베이스의 모든 부분에서 작업을 쉽게 시작할 수 있도록 일관된 스타일을 유지합니다.
blockly-samples 및 core Blockly에 포함되는 모든 코드는 커뮤니티 참여자 또는 Blockly 팀 구성원이 작성했는지 여부를 검토합니다.
Google은 검토자와 협력하여 변경사항을 최대한 원활하게 적용할 수 있도록 노력하고 있습니다. 참여자는 검토를 통해 pull 요청을 받고 병합할 수 있도록 Google과 대화에 참여해 주시기 바랍니다.
처리 과정
PR 검토 절차는 다음과 같은 단계를 거칩니다.
임무
pull 요청이 접수되면 Blockly팀의 대기 중인 구성원이 검토자를 할당합니다.
검토자는 전문성을 토대로 선정되며 워크로드를 균등하게 분배합니다.
검토자가 할당되기까지 며칠이 걸릴 수 있으며 검토를 받으려면 며칠이 더 걸릴 수 있습니다. 이는 정상적인 현상이므로 걱정하지 않으셔도 됩니다.
의견
의견 단계에서 검토자가 PR에 대한 변경사항을 제안합니다. Google JavaScript 스타일 가이드를 준수하도록 코드를 작성하는 간단한 작업일 수 있습니다. 함수 정의를 재구성하도록 요청하는 것과 같이 더 큰 규모의 작업일 수도 있습니다.
검토자는 여러 알림이 아닌 단일 알림을 받을 수 있도록 개별 주석을 작성하는 대신 GitHub의 코드 검토를 사용하는 것이 좋습니다.
토론
토론 단계는 의견에 응답할 수 있는 기회입니다. 리뷰 의견 중 하나가 명확하지 않았을 수도 있습니다. 이제 확인을 요청할 수 있습니다. 또는 검토자가 변경을 요청했지만 부정적인 영향을 미칠 것이라고 생각했을 수 있습니다. 바로 지금이 절충안을 찾을 기회입니다.
버전
수정 단계에서는 PR을 변경합니다. 일반적으로 이러한 변경은 의견 제출 단계에서 검토자가 말한 내용으로 인해 발생합니다.
수정을 완료한 후 검토자에게 태그를 지정하여 다시 검토해 달라고 요청하는 것이 좋습니다.
반복
수정 단계 후 검토자가 의견을 제공할 수 있는 기회가 다시 제공되며, 처음부터 절차가 시작됩니다.
두 번째 검토는 종종 간단하며 구두점 및 코드 스타일과 같은 니트에 중점을 둡니다. 하지만 때로는 두 번째 검토가 상당히 중요할 수 있습니다. 첫 번째 리뷰어는 다른 사람에게 새로운 관점을 가지기 위해 리뷰를 요청할 수도 있습니다.
병합!
합병 단계는 기념할 기회입니다. 하나의 변경사항을 만들고 논의 및 수정한 후 최종적으로 병합했습니다. 이는 많은 사람이 완료하지 않을 뿐 아니라 시작도 하지 않는 웅장한 업적입니다.
Blockly를 개선하기 위해 노력해 주셔서 감사합니다. 그리고 축하합니다.