Pull Request 리뷰
Source: Dev.to
PR 리뷰가 필요할까요?
당연합니다.
2단계 리뷰 프로세스
Software Engineering at Google에서는 2단계 리뷰 프로세스를 다음과 같이 설명합니다:
- 언어 리뷰 – 언어 사용에 초점을 맞추어 일관성과 공유된 의미 체계를 보장합니다.
- 기능 구현 리뷰 – 기능의 정확성과 설계에 대해 검토합니다.
때로는 같은 사람이 두 단계를 모두 담당할 수 있습니다(예: 파이썬 승인자 그룹의 일원인 엔지니어).
목표는 코드베이스 전체에서 언어 사용을 일관되게 유지하고, 자동화된 린팅 도구를 보완하는 것입니다.
인간 리뷰의 어려움
구글 엔지니어가 한 번은 PR이 한 달 동안 보류됐다고 보고했습니다. 설계 단계에서 요청된 피드백은 대부분 무시되었지만, PR 단계에서는 많은 리뷰어가 매우 구체적인 의견을 제시했습니다. 이는 타이밍과 기대치가 마찰을 일으킬 수 있음을 보여줍니다.
인간이 어려워하는 부분을 자동화하기
Lint, Prettify 등
GitHub을 사용한다면 GitHub Actions를 설정해 린팅, 포맷팅(예: Prettier), 타입 체크(동적 타입 언어의 경우)를 자동화할 수 있습니다. 반나절 정도의 작업으로 다음을 할 수 있습니다:
- 액션 추가,
- 필수 체크로 지정,
- 메인 브랜치 보호.
판단이 필요한 결정(예: 파이썬에서 리스트 컴프리헨션을 피해야 할 때, 함수가 “하나의 일”을 technically 수행하더라도 분할해야 할 때)에는 스타일 가이드를 채택할 수 있습니다. 이러한 관행을 문서화하면 매 PR마다 반복되는 논의를 방지할 수 있습니다.
테스트와 테스트 커버리지
PR 체크에 다음을 추가하세요:
- 테스트 실행 – PR은 모든 테스트를 통과해야 합니다.
- 커버리지 메트릭 – 코드베이스가 테스트에 의해 얼마나 커버되는지 추적합니다.
초기에는 테스트 실패 시에만 PR을 실패시키고 커버리지 데이터를 수집합니다. 이후 낮은 커버리지 임계값을 도입하고 점진적으로 높여갑니다. 이 자동화는 명백한 정확성 문제에 대한 수동 리뷰 필요성을 줄여줍니다.
수동 리뷰가 되어서는 안 되는 것
많은 조직에서 자동화 도구가 부족하면 “브랜치가 보호되어야 하며, 누군가가 PR을 승인해야 한다”는 규칙이 생깁니다. 자동화 설정은 보통 며칠이면 끝나지만, 익숙하지 않거나 빠르게 배포하고 싶어하는 이유로 건너뛰는 경우가 많습니다. 자동화를 대신해 인간 게이트를 두는 것은 배포 속도를 높이는 것과 정반대입니다.
수동 리뷰는 모든 자동화 가능한 것을 포괄하는 역할을 해서는 안 됩니다. 체크를 자동화할 수 있다면 자동화하세요; 코드 기여는 애플리케이션의 생명줄이므로 흐름을 향상시키는 자동화를 우선시해야 합니다.
수동 리뷰가 집중해야 할 것
수동 리뷰는 무형의 영역에 속합니다—작성자에게 사고 파트너가 될 기회입니다. 전 동료인 사이먼이 이렇게 말했습니다:
“당신은 작성자에게 훌륭한 답변을 얻도록 돕기 위해 맥락, 경험, 조언을 제공하고 있습니다… 이는 정확성에 관한 것이 아니라, 작성자가 고려하지 않았을 수 있는 트레이드‑오프를 안내하고, 최종 결정을 내릴 수 있게 empower 하는 것입니다.”
이는 Dawna Markova가 *Collaborative Intelligence*에서 제시한 “Thinking Partners” 개념과 일맥상통합니다.
리뷰어를 위한 안내 질문
- 변경 사항이 의도한 트레이드‑오프를 반영하고 있나요?
- 코드가 읽기 쉽고, 유지보수가 용이하며, 미래의 기여자에게 친절한가요?
- 복잡성과 정확성 사이의 균형을 잘 맞추고 있나요?
이러한 논의는 요청이 아니라 교환이며, 작성자가 자신의 결정을 확인하거나 재평가하도록 돕습니다.
리뷰어가 서브시스템에 대한 지식, 사용자 피드백, 지원 경험 등 역사적 맥락을 제공하면 리뷰는 강력한 지식 공유 순간이 됩니다. 반대로 비즈니스 맥락이 부족한 리뷰어는 숨겨진 가정(예: “우리는 항상 이렇게 해왔다”)을 드러내어 작성자가 재고하도록 만들 수 있습니다.
더 넓은 영향
신중하게 진행된 PR 리뷰는 다음과 같은 매개체가 됩니다:
- 지식 공유,
- 조직 전체의 역량 강화,
- 관계 구축 및 의사결정 능력 향상.
게이트키핑에서 협업적 개선으로 전환되는 것입니다. 여러분의 경험은 다를 수 있으며, 그것도 가치 있습니다—자신의 관점을 자유롭게 공유하세요. 핵심은 PR 리뷰를 사후적인 것이 아니라 의도적이고 전략적인 실천으로 다루는 것입니다.