Pull Request 규칙

• 코어타임 시, PR 날리면 최대 1시간 이내에 코드리뷰
• PR은 아무 때나 올리되, 위 조항은 코어타임에만 해당
• PR 리뷰어는 최소 2명 (2명 승인했다고 바로 머지하지 말기, 나머지 한 명한테 물어보기)
• PR 코멘트 남길 때 레퍼런스 있다면 같이 첨부 권장
• 저돌적인 코드리뷰 적극 권장
• 트러블 슈팅 적극적으로 하기 (노션에 적고 관련 PR에 첨부)
• 구현하다 잘 안 되면 질문 찬스 활용하기
• Pn 방식 코드리뷰 하기 (P1 ~ P3)
  • P1 : 수정하지 않으면 Approve 안해줌 ~
  • P2 : 코드에 대한 타당한 근거가 존재하는 경우 Approve 가능 ~, 아니면 빠꾸..
  • P3 : 간단한 의견사항 (반영 안해도 됨) → 읽었다는 의미에서 따봉정도는 해주는 걸로 👍
• 커밋, PR 잘게 쪼개기
  • 일단 300줄 내외?
  • 일주일 해보고 회고시간에 다시 정해보자 .ᐟ.ᐟ
• PR Merge 하고나면 브랜치 삭제하기