PR merge 승인의 최소 기준에 대한 논의

[rmkim7]

저희가 보통 PR을 작성할 때 리뷰어를 팀 전체로 설정하게 될 것 같은데
모든 팀원들이 리뷰를 한 뒤에 PR merge를 승인하면 좋겠지만,
모두가 시간에 맞춰 리뷰를 하는 것을 기다리다 보면
작업에 병목현상 이 생길 수도 있을 것 같습니다.

이런 상황에 대비하여 PR merge 승인의 최소 기준을 미리 정해놓으면 어떨까요?
제가 생각한 최소 기준의 예시는 아래와 같습니다.

• 같은 팀의 다른 팀원 1명 + 리뷰어 1명

제가 작성한 예시는 인원에 대한 기준이지만
다른 부분에 대한 기준을 제시해주셔도 좋고
최소 인원에 대한 다른 기준을 제시해주셔도 좋을 것 같습니다.

다른 분들의 의견이 궁금합니다.
편하게 피드백 부탁드립니다😀

[dgd03146]

현재 샘플 프로젝트는 규모가 매우 작기때문에 한명의 리뷰어로 해도 충분할 것 같고 나중에는 정아님이 말씀해주신것처럼 2명의 리뷰어를 최소로 해서 하면 적합할 것 같습니다.

[Seono-Na]

PR 승인으로 인해 병목이 생기는 걸 짚어주셔서 감사합니다.
PR 승인 기준은 저희가 협업하는데 있어서 중요한 부분이라 생각하기 때문에
@rmkim7 님 의견에 동의하는 부분입니다.

그래서 PR 승인에 있어서 인원과 그 인원 구성이 중요하다고 생각되는데요,
말씀해주신 대로
함께 작업하는 팀원 1명과 리뷰어 중 1명을 최소 구성 인원으로
최소 2명은 참여하여 PR merge 승인을 하는 것이 좋아 보입니다!

다른 분들의 의견도 궁금합니다..🙂

[choi1five]

PR merge 승인 기준 및 코드 리뷰 방식 제안 💭

PR 승인 기준은 협업하는데 있어서 중요한 부분이라 생각하기 때문에 좋은 제안을 해주셔서 감사합니다! 🙏

본 프로젝트 진행 전 코드리뷰 문화 자체를 경험하는 부분도 있다보니 @Seono-Na 님의 의견대로 함께 작업하는 팀원 1명과 리뷰어 중 1명으로 총 2명을 최소 구성 인원으로 진행하면 좋지 않을까 싶습니다. ✨

---

더불어 코드 리뷰를 어떻게 작성할지에 대한 부분도 같이 논의를 해보면 어떨까요?

코드 리뷰 과정에서 기능 동작에 대한 테스트도 중요하지만, 코드 자체에 대한 의견을 주고받는 것도 중요하다고 생각합니다.
저희가 아직 좋은 코드와 나쁜 코드를 구별하는 경험이 부족할 수 있지만, 이는 오히려 더 적극적인 토론의 기회가 될 수 있을 것 같습니다. 💡

본인이 작성한 코드와 다른 방식으로 구현된 코드를 "제 코드는 이런 방식인데, 각각 어떤 장단점이 있을까요? 🤔"와 같이 서로의 코드에 대해 배우고 성장할 수 있는 대화를 나눠보면 좋을 것 같습니다!!!