-
Notifications
You must be signed in to change notification settings - Fork 2
PR 코드리뷰 Rule✍
이도현 edited this page Jul 11, 2023
·
1 revision
-
Merge 규칙
- 각 파트 전원의 Approve를 받아야 한다.
-
코드 리뷰 시 커멘트 분류
- 간단한 의견 및 질문: 문장의 앞에 💬 (speech balloon)을 붙인다. (ex. 💬 아코 이거 의견이 궁금해요~)
- 변경 제안 및 토론이 필요한 내용인 경우, 그냥 내용을 쓴다. (ex. 아코 filter를 사용하는건 어떨까요~?)
- 변경 제안이 있는 경우: Request Change 선택 / 변경 제안이 없는 경우: Approve
-
리뷰 내용 규칙
-
리뷰에 대한 토론은 오프라인으로 대화한다.
- 리뷰를 이해하지 못한 경우, 토론이 필요한 경우에는 대화를 한다.
- 대화의 결론은 코멘트로 남겨둔다.
-
작업, 리뷰 반영 기간
- 작업 : 리뷰 및 반영 2 : 1의 비율로 진행한다.