우리 회사의 여러 팀이 전에는 본 적이없는 코드 검토 워크 플로를 연습합니다. 회사 전체의 일관성을 유지하는 데 가치가 있다는 생각으로 그 배후의 생각을 이해하려고합니다. (여러 코드베이스에 기여하고 과거의 차이점으로 인해 넘어졌습니다.)
- 코드 작성자가 풀 요청을 제출
- 검토자가 코드를 검사
- 검토자가 승인하면 "좋아요, 자유롭게 병합하십시오"라는 문구를 따라 의견을 남깁니다.
- 검토자가 우려 사항이있는 경우 "사소한 문제 X 및 Y를 수정 한 다음 병합하십시오"와 같은 주석을 남깁니다 (주요 변경 사항은 2 단계로 돌아가십시오).
- 코드 작성자는 필요한 경우 변경 한 다음 자신의 풀 요청을 병합합니다.
다음과 같은 우려가 있습니다.
3 단계에서 승인하는 경우이 워크 플로우는 끌어 오기 요청 작성자에게 불필요하게 보이는 왕복을 생성합니다. 이미 코드를보고있는 검토자는 즉시 코드를 병합 할 수 있습니다.
3 단계에서 변경 요청이있는 경우 이제 풀 요청을 병합하는 기관은 전적으로 PR 작성자에게만 달려 있습니다. 작성자 외에는 병합 전에 변경 사항을 보지 않습니다.
이 워크 플로우의 다른 장점이나 단점은 무엇입니까? 이 워크 플로는 다른 엔지니어링 팀에 공통적인가?