-
Notifications
You must be signed in to change notification settings - Fork 3
PR & Code Review Convention
Inho Choi edited this page Dec 7, 2022
·
5 revisions
-
Title: 문장형으로 작성 부탁드립니다.
-
Description: PR Template를 참고 바랍니다.
- 아래의 라벨링을 코멘트 앞에 붙여주세요.
Label | Review | Mean | Description |
---|---|---|---|
[P1] | Request changes | 꼭 반영해주세요. | 반영할 수 없다면 합리적인 의견을 통해 리뷰어를 설득할 수 있어야 합니다. |
[P2] | Request changes | 적극적으로 고려해주세요. | 수용할 수 없다면 적합한 의견을 들어 토론을 권장합니다. |
[P3] | Comment | 웬만하면 반영해 주세요. | 수용할 수 없다면 반영할 수 없는 이유를 설명하거나 Issue를 만들어서 다음에 반영할 것을 표현합니다. |
[P4] | Approve | 반영해도 좋고 넘어가도 좋습니다. | 의견을 달지 않고 이모지로 반응을 남겨도됩니다. 의견을 반영하는게 좋을지 고민했다면 충분합니다. |
[P5] | Approve | 그냥 사소한 의견입니다. | 작성 및 이모지를 남기지 않아도 충분합니다. |
[의견] | Approve | 궁금함 혹은 정말 개인적인 질문들 |