code review는 TL의 가장 leverage가 높은 활동 중 하나입니다. 품질을 보호하고 가르칩니다. 목표는 더 나은 코드와 더 강한 팀이지, gatekeeping이 아닙니다. 리뷰는 빠르고, 친절하며, 중요한 것에 집중해야 합니다.
원칙
text
✓ 코드를 리뷰하라, 절대 사람이 아니라("you"가 아니라 "this function")
✓ 반드시 고쳐야 할 것과 있으면 좋은 것을 구분하라(nit은 명시적으로 표시)
✓ 명령하지 말고 질문하라("X는 어떨까요?"는 논의를 유도한다)
✓ 문제뿐 아니라 좋은 작업도 칭찬하라
✓ 빠르게 하라 — 이틀간 막힌 PR은 모멘텀을 죽인다
✓ 완벽할 때가 아니라 충분히 좋을 때 승인하라
구체적인 예시
"이건 틀렸어요, map을 쓰세요" 대신 이렇게 쓰십시오. "nit: 여기서 map을 쓰면 중첩 루프를 피할 수 있는데, 그럴 가치가 있을까요? 블로킹은 아닙니다." 같은 요점이지만, 가르치고, 반대할 여지를 남기며, merge를 막지 않습니다.
권력 역학을 주의하라
TL로서 당신의 코멘트는 추가적인 무게를 지닙니다. 무심한 "저라면 다르게 하겠어요"가 명령처럼 느껴질 수 있습니다. 무엇이 진짜 우려이고 무엇이 선호인지 명확히 하고, 사람들이 반박할 수 있게 하십시오.