-
Notifications
You must be signed in to change notification settings - Fork 3
PR & Code Review Convention
Inho Choi edited this page Oct 18, 2022
·
5 revisions
Label | Review | Mean | Description |
---|---|---|---|
[P1] | Request changes | 꼭 반영해주세요. | 반영할 수 없다면 합리적인 의견을 통해 리뷰어를 설득할 수 있어야 합니다. |
[P2] | Request changes | 적극적으로 고려해주세요. | 수용할 수 없다면 적합한 의견을 들어 토론을 권장합니다. |
[P3] | Comment | 웬만하면 반영해 주세요. | 수용할 수 없다면 반영할 수 없는 이유를 설명하거나 Issue를 만들어서 다음에 반영할 것을 표현합니다. |
[P4] | Approve | 의견을 달지 않고 이모지로 반응을 남겨도 충분합니다. 해당 의견을 반영하는게 좋을지 고민해 보는 정도면 충분합니다. | |
[P5] | Approve | 작성 및 이모지를 남기지 않아도 충분합니다. |