Pull Request Convention

• Pull Request(이하 PR) Template을 적극 활용한다.

• PR은 "[타입] 제목"의 형태로 작성한다.

• 하나의 PR에는 모든 commit을 합쳐 추가된 라인이 250 라인을 넘기지 않도록 한다.

• PR Template 최하단의 Close를 사용하여 Issue를 연결해 PR이 완료될 때, Issue가 같이 닫히도록 한다.

• 제목에는 작성한 코드의 내용을 한 줄로 요약한다.

• PR을 요청한 후에는 merge를 기다리지 않고 다른 브랜치를 추가하여 계속 작업한다.

• 모두의 Approve를 받은 후에 merge한다.

• merge는 PR의 Assignee가 한다.

  • 단, !HOTFIX의 경우 모두의 Approve를 충족한 후에 1시간 이상 merge되지 않았을 경우 Assignee가 아닌 다른 사람이 merge할 수 있다.

• 서로 승인하지 않은 상황에서 함부로 merge하지 않는다.

• 올라온 PR에 대해서 반드시 최대 12시간 안에 확인하도록 한다.

• !HOTFIX가 붙은 PR은 최대 1시간 안에 확인하도록 한다.

• !HOTFIX가 아닌 PR 중 빠르게 코드 리뷰 및 merge되어야 하는 경우 타입 앞에 !를 붙여서 표시하고, label에 !fast를 같이 표시한다.

• !HOTFIX를 제외한 !로 시작하는 PR은 최대 3시간 안에 확인하도록 한다.

• 정해진 시간동안 PR에 대해 리뷰하지 않은 경우, 수시로 다른 팀원들에게 리뷰를 요청할 수 있다.