5. 코드리뷰와 리뷰 반영

[JINi0S]

1. 코드리뷰

• 리뷰어는 리뷰를 남기고, PR에 대해 3가지 의사표현을 할 수 있다.
  • Comment : 그냥 코멘트만 달아줌.
  • Request changes : 코드에서 버그를 발견하면 다시 수정해달라고 요청할 수 있음.
  • Approve : 이 코드가 merge 되는 것에 동의함.
• 본인 제외 1명에게 Approve를 받아야 PR을 완료할 수 있다.
  • 코드에 수정이 필요없다면 Comment 작성 후 Approve 한다.
  • 만약 코드에 수정이 필요하다면 Comment만 달거나, Request change를 요청할 수 있다.
    이 경우에는 Merge 승인이 되지 않는다.
• 코드리뷰 내용 반영한 후 해당 리뷰의 커멘트에 커밋 id 남기기
• 리뷰 내용을 확인했으면 커멘트에 이모지로 확인했음을 알리기

2. 이슈 반영(Merge)

Approve 리뷰를 받은 코드에 대해 PR 작성자가 직접 merge 한다.

[HIBAR97]

코드 수정이 필요없다면 따로 Comment없이 Approve만 하는것으로 이해하면 될까요?

[JINi0S]

코드에 수정이 필요없다면 Comment 작성 후 Approve 한다.
이렇게 작성해두긴 했는데 커멘트 없이 해도 좋을 것 같습니다! 커멘트 없이 하는 걸로 수정할까요??
@HIBAR97

[HIBAR97]

제가 누락했군요...
간단한 커멘트 1개라도 있으면 괜찮을 것 같아요!