dev 브랜치 PR 정책을 문서화해 팀 협업 신뢰도를 높였다
목차
PROGRESS.md를 수정해서 브랜치 흐름을 정정했다. "slecs 작업도 dev에 직접 커밋하면 안 되고, dev PR을 통해야 한다"는 룰을 명시한 것인데, 사실 이건 이미 어느 정도 지켜지고 있던 관행을 문서로 명확히 한 거다.
왜 이제서야 명문화했나
팀이 작을 때는 "뭐, 알아서 다들 하겠지" 했는데, 팀원이 늘고 일하는 방식이 다양해지니까 문제가 생겼다. 누군가는 "dev에 직접 푸시해도 되겠지?" 했고, 누군가는 "항상 PR이 필수인 줄 알았는데" 했다. 특히 내부 작업이라고 생각되는 "slecs 작업" 같은 경우, 그냥 dev에 바로 집어넣어도 되지 않을까 라는 생각을 하는 사람들이 있었다.
하지만 "슬렉스 작업이니까 좀 더 편하게" 하려다 보면, 어느 순간 누가 뭘 왜 고쳤는지 추적하기 어려워진다. 리뷰 없이 들어온 코드들이 섞이면서, 나중에 버그가 터졌을 때 원인 파악이 지연된다. 또 dev 브랜치에 직접 커밋이 쌓이면, 다음 리리즈 준비할 때 "이 커밋들이 정말 test된 건가?" 하는 의심이 생긴다.
프로세스는 느리지만, 신뢰는 올라간다
결국 이 정책은 "속도 vs 신뢰성" 트레이드오프에서 신뢰성을 선택한 거다. PR을 거친다는 건:
- 최소한의 눈 2개: 작성자 + 리뷰어가 코드를 본다
- 의도 기록: PR 제목, 설명, 댓글에 "왜 이렇게 했는지"가 남는다
- CI 재확인: PR마다 자동 테스트/린트가 다시 한 번 돈다
- 협업의 증거: git history에 approve 마크가 남는다
내가 리더로서 느낀 게, 작은 fixes나 typo 수정도 PR을 거치면, 팀원들이 "어라, 이건 이렇게 하는 거구나" 배우게 된다. 심지어 내 코드에 대한 리뷰 받으면서도 "아, 내가 놓친 부분이 있네" 하는 경험을 자주 한다.
| 정책 | direct commit to dev | dev PR required |
|---|---|---|
| 속도 | 빠름 | 약간 느림 (리뷰 시간) |
| 추적성 | 약함 | 강함 (PR 히스토리) |
| 리뷰 강제성 | 없음 | 있음 (merge 조건) |
| 문제 발생 시 원인 파악 | 어려움 | 쉬움 (PR 코멘트 남음) |
| 신입/팀원 학습 | 낮음 | 높음 (리뷰 코멘트) |
문서화의 역할
이 정정 커밋의 핵심은 사실 "새로운 규칙"이 아니라, 모호했던 규칙을 명확히 한 것이다. PROGRESS.md 같은 곳에 "slecs 작업도 dev PR을 통해야 한다"고 쓰는 순간, 새로운 팀원이 들어와도 온보딩이 쉬워진다. Slack 메시지나 구두로 "원래는…" 설명할 필요가 없다.
실제로 문서가 없으면:
- 팀원A: "slecs는 내부 작업이니까 가볍게 가자"
- 팀원B: "아뇨, 저는 항상 PR 올렸는데?"
- 나: "앗, 우리가 규칙을 명시하지 않았네…"
요 상황이 반복된다.
회고: 프로세스는 팀 크기에 따라 진화한다
처음에 2-3명일 때는 git 규칙이 필요 없었다. 하지만 팀이 5명, 10명으로 커지면서, 각자의 "좋은 의도"가 충돌하기 시작한다. 누군가는 "속도가 최우선"이고, 누군가는 "품질이 최우선"이다. 그럼 프로세스가 그 중간을 찾아줘야 한다.
내 배운 점은: 프로세스는 제약이 아니라, 신뢰를 위한 공동 언어라는 것. PR 문화가 정착되면, 팀원들이 "미안해, 내가 실수했어" 보다 "이 방향 어때? 리뷰 부탁해"라고 말한다. 그게 건강한 협업의 시작이다.
🛒 이 글과 어울리는 추천 상품
*위 링크는 쿠팡파트너스 활동의 일환이며, 일정액의 수수료를 제공받을 수 있습니다.
댓글 0
첫 댓글 달아줘.