공통 ‐ 팀의 문화 원칙 - ConSeat/frontend GitHub Wiki
📃 코드 리뷰 원칙
- push를 한다면 workflow가 작동하여 자동으로 PR이 생성돼요. 이때 디스코드 알림이 오는데 PR이 작성되어 있지 않다면 개발을 끝내지 않은 것이니 연락이 올 때까지 코드 리뷰는 지양해주세요.
- PR 작성을 끝낸 후 팀원들에게 코드 리뷰를 요청해요. 이땐 Freezing이 원칙이에요. 추가 커밋을 하려면 팀원들에게 말해주세요. 코드 리뷰 중 코드가 변경되면 팀원들이 헷갈려 해요.
- PR 본문을 보고 코드 리뷰를 시작해요. 말투는 친절하게 내용은 날카롭게! 컨벤션과 버그 가능성 위주로 보고, 궁금하면 질문해요.
- 꼭 모든 리뷰에 반응을 해주세요! 질문이 아니라면 이모지만 달아도 괜찮아요. 피드백을 반영하고 싶지 않다면 그 이유를 적어주세요.
- 코드 리뷰 후 피드백을 반영하여 추가 커밋이 발생했다면 코드 리뷰를 재요청해 주세요. 확인하지 않은 내용이 추후 개발 중에 발견되면 문제가 생길 수 있어요.
- 승인이 되었다면 머지를 해도 되는지 물어보고 머지를 해요.