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) 이거 왜 이렇게 했나요?)