지금 SF 개발자 커뮤니티에서 화제가 되고 있는 레포지토리 중 하나가 방금 5K 스타를 넘어섰어요.

그리고 저는 PR(Pull Request) 리뷰에 대해 무언가를 깨닫게 되었죠:

우리는 diff(변경 사항)을 검토하는 데 너무 많은 시간을 쓰고 있고,
실제로 그 diff가 무엇을 망가뜨릴 수 있는지 질문하는 시간은 충분하지 않아요.

모든 PR에는 놓치기 쉬운 세 가지가 있어요.

1. 변경 사항 옆에 있는 테스트가 당신의 변경 사항이 테스트되었다는 것을 의미하지는 않아요.
   그것은 단지 테스트가 존재한다는 것을 의미할 뿐이에요.
2. 이 둘은 다른 거예요.
3. 10줄짜리 PR도 10,000줄짜리 PR과 정확히 동일한 다운스트림(downstream)의 8개 파일을 망가뜨릴 수 있어요.

그래서 'PR을 작게 유지하세요'는 정말 답이 아니에요.
크기 자체가 문제가 아니었어요.
문제는 도달 범위(reach)였어요.

도구는 Graft예요.
이것은 당신이 변경한 것에서 실제 의존성 그래프를 추적해요.
단지 diff에 있는 파일들만 추적하는 게 아니라요.
의존성을 증명하는 정확한 라인을 보여주고,
터치되지 않은 다운스트림 테스트들을 플래그 지정하며,
당신의 변경 사항이 실제로 도달하는 코드를 기반으로 리뷰어들을 추천해 줘요.

제가 흥미롭다고 생각하는 부분이 바로 이거예요.

'이 변경 사항이 실제로 무엇에 영향을 미칠 수 있을까?'라는 질문에 답한다는 거예요.
대신에 '어떤 파일들이 변경되었니?'라고 묻는 대신 말이죠.
답글로 남기면서요.