제목 : [브랜치 이름] 작업내용 간단하게 ex) [feature/api] api 작업 ## #️⃣연관된 이슈 > ex) #이슈번호, #이슈번호 ## 💡 핵심적으로 구현된 사항 <-- 문제를 해결하면서 주요하게 변경된 사항들을 "스크린샷"과 함께 적어 주세요 --> ## ➕ 그 외에 추가적으로 구현된 사항 <-- 없으면 "없음" 이라고 기재해 주세요 --> <-- 주 Task 이외의 작업한 변경 사항 --> ## 🤔 테스트,검증 && 고민 사항 <-- 배포에서 체크해봐야 할 부분 --> <-- 궁금한 점, 팀원들의 의견이 필요한 부분, 크로스체크가 필요한 부분 등 -->
PR 에 대한 리뷰는 작성자가 팀원을 2명을 rotation으로 정해서 진행
- 코드리뷰 때문에 Merge가 밀리는 상황을 방지하기 위하여 특별한 일이 있지 않는 이상 매일 코드 리뷰 시간을 다 같이 가집니다!
반드시 Slack 에 사전 공지를 통해 알립니다. 🙋♂️
- 모든 리뷰는 서로를 배려하며! 이해가 안가는 부분은 반드시 작업자에게 질문!
- 현재
Github Actions를 통해 리뷰어를 랜덤으로 2명 할당 - 본인이 직접 지정하지 말기!(직접 지정하게 되면 로직이 꼬임)
Assignee도 자동으로 본인이 할당되기 때문에 이것 또한 본인이 직접 지정하기 말기
📌 PR Comment 작성 시 Prefix for Reviewers
* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
Ex) P1 : ~코드 보안적으로 취약해서 제거해야 할 것 같습니다
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
Ex) P2 : 이 방식으로 하면
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)
* ASK : 궁금한거 질문(ex) 이거 왜 이렇게 했나요?)