[멘토링] 협업 간 코드리뷰 관련 질문 (답변희망멘토 : jayden.bin)

[codeok2]

🙋질문자

판교 3기 / 5조(개발자지만 독서는 하고싶지) - bruni.kim(김범석)/풀스택

🙋답변 희망 멘토

jayden.bin(김한빈)/@konempty

✏️ 궁금한 내용

프로젝트에서 백엔드 인원이 두 명이라 서로 코드 리뷰를 진행하면서 개발을 해나갈 예정입니다. 하지만 팀원 둘 다 코드 리뷰 경험이 없는 상황이라 개발 속도에 영향을 끼칠 것 같아 걱정입니다.

[추가 이해를 위한 디테일한 질문]

• 코드 리뷰 경험이 없는 주니어 개발자들이 효과적으로 코드 리뷰를 시작하려면 어떤 방식으로 접근하는 것이 좋을까요?
• 개발 속도와 코드 품질 사이에서 적절한 균형을 맞추려면 코드 리뷰를 어느 정도의 볼륨(depth)으로 진행해야 할까요?
• 모든 PR을 리뷰하는 것이 부담스러운 상황에서, 어떤 기준으로 리뷰할 PR을 선택하면 좋을까요?

👀 시도한 방식

PL 미팅에서 다음과 같은 의견을 받았습니다:

• 코드에 대한 질문 중심으로 접근하기 (예: "이 코드 왜 이렇게 작성했나요?", "저였으면 이렇게 작성했을 것 같은데 어떻게 생각하세요?")
• 모든 PR에 대해 리뷰하려고 하지 말고, PL께 특정 PR에 대해 리뷰를 진행할지 여쭤보고 선택적으로 진행하기

⏭️ 이제 어떻게 하려고 하는지?

구체적으로 어떤 기준과 프로세스로 진행하면 좋을지 멘토님의 조언을 얻고 PL께 받았던 의견과 취합하여 바로 적용해볼 예정입니다.

[konempty]

@codeok2
안녕하세요 브루니~!
두분다 코드리뷰 경험이 없다면 험난한 길이 되겠네요 ㅎㅎ..

일단 초반에는 두분의 합이 잘 맞지 않을테니 주로 개발 컨벤션등에서 많은 리뷰가 오갈거예요
따라서 저는 개발 컨벤션을 초반에 잘 잡아 두어야 이후 PR 리뷰에 들어가는 시간을 줄일수 있다고 생각해서
새로운 사람들과 개발을 할때는 개발 컨벤션 부터 단단히 잡아두고 시작하는 편이예요.

컨벤션으로 인한 리뷰가 줄어 들게 되면 그때서야 순수하게 기능에 대한 리뷰만 진행할수 있을텐데
지금은 시간이 촉박하니 이것도 최대한 줄이고 싶어하시는게 느껴지네요 ㅋㅋㅋ

요기에서 저희 부서가 신입에게 해주는 조언을 해드리면 좋을것 같은데
저희 부서에선 PR이 머지되기전의 코드는 "담당자의 코드"라고 하구요 PR이 머지되면 "우리 모두의 코드"라고 해요
따라서 PR이 머지되기전에 코드에서 문제가 생기면 담당자가 보통 책임을 지고 해결하지만
PR이 머지되고 난 이후에는 "누구 하나의 책임이 아닌 모두가 책임을 나눠가지고 다 같이 해결한다" 라고 해요

따라서 제가 추천하는건 물론 되도록 모든 PR을 확인하는거지만 그게 어렵다면

1. "이번 작업은 온전하게 내가 보증할수 있음" 이라고 작업한 당사자가 생각 한다면 믿고 간다
2. "이 작업에 대해 내가 온전히 책임지긴 힘듦" 이라고 생각한다면 그건 PR 리뷰가 필요하겠죠
   따라서 작업한 당사자가 해당 작업에 대해 얼마나 자신 있는지에 따라 나눠서 리뷰 한다면 좋을것 같아요.

반대로 위와 같은 _정성적인 지표_가 아닌 정량적인 지표(변경된 라인수, 파일수)로 구분할수 있지 않을까? 라는 의문이 드실수도 있는데
코드가 10줄만 수정했어도 크리티컬한 변경사항일수도 있구요
파일이 20개가 변경되었어도 단순 반복노동이라 큰 문제가 예상 되지 않을수도 있기 때문에
정량적인 지표로 딱 떨어지는 기준을 세우기는 어려울거라고 생각해요

마지막으로는 PR을 깃헙에서 각자 리뷰하는것도 좋지만 저는 가끔씩은 두분이서 시간을 잡고 작업자가 자신의 PR에 대해서 설명해주면서
왜 이렇게 작업 했는지 의도와 방향성을 설명해주는 시간을 가지는것도 좋다고 생각합니다!

[codeok2]

답변 감사합니다 제이든!
작업자의 자신감을 기준으로 PR 리뷰 결정을 한다는 점이 생각지도 못한 방식이라 신선하고 그만큼 PR 리뷰를 요청하지 않기 위해서는 담당자로써 더 확실한 근거를 가지고 작업을 해야겠다는 생각이 드네요! 저는 코드 리뷰 여부 기준을 결정할 때 핵심 비즈니스 사항인가 아니면 단순한 기능 추가인가와 같이 기능별로 나누는 걸생각했습니다!
제이든의 답변을 참고하여 작업할 때 내 자신이 보증할 수 있을만큼 작업을 진행하되, 조금이라도 애매한 부분이 있으면 해당 부분에 대해서만 코드리뷰를 요청하는 것도 시간을 최대한 아끼면서 코드리뷰를 진행할 수 있는 방법이 될 것 같다고 생각이 드네요!

팀원분과 해당 내용에 대한 회의 이후 곧바로 적용해보겠습니다!
다시 한 번 답변 감사합니다.👍