Pull request
여러 사람과 협업에 있어서 main(master) 브랜치에 무분별한 접근과 수정을 제한하기위해 Pull request를 사용합니다.
Branch protection rule 설정은 레포지터리 권한이 Maintain or Admin 인 경우 가능합니다.
index 바로가기
Branch protection rule 설정하기
다른 사람의 Pull requst 리뷰하기
1. 레포지터리의 Setting 탭 -> 좌측 사이드바 Branches -> Branch protection rule에서 할 수 있습니다.
`Branch protection rule`은 다양하지만, 해당 프로젝트의 경우
-
Require a pull request before merging
-
Require approvals
-
Require conversation resolution before merging
세 가지를 채택하고있습니다.
이해하기 쉽게 사진과 함께 설명해보겠습니다.
현재 Merging is blocked 즉, 머지할 수 없는 상황입니다.
위에서부터 하나씩 보겠습니다.
Review required - At least 3 approving reviews are required by reviewers with write access
Branch protection rule에서 인원을 3으로 설정해서 그렇습니다.
현재 JJinuk 이외에는 review&approve가 없어서 머지할 수 없습니다.
우측 상단에 보이는 Add your review 혹은 File changed 탭으로 이동합니다
리뷰 할 소스코드의 + 버튼을 누르면 됩니다.
Add single comment - 말 그래도 1개의 코멘트만 작성합니다(review)
Start review 이후 코멘트는 Pending이 되며, 우측 상단 Finish your review 누르면 일괄적으로 코멘트가 작성됩니다.
만약 Finish your review 누르지않는다면, Pending 된 코멘트들을 표출되지않습니다.
많이들 헷갈리는 포인트입니다.
우리 룰은 review&approve 였습니다.
여기서 comment로 작성하게 되면 approve 되지않기에 재차 approve를 해주어야합니다.
혹시나 코드의 수정이 필요한 부분은 request changes를 해야하며, 꼭 resolve problem 버튼을 눌러줘야합니다.