Git PullRequest

병합 과정에서 다른 사람들의 코드 리뷰 혹은 검토 받고 싶을 때 사용하거나 권한이 없는 저장소에 대해 본인의 코드를 병합하고 싶을 때 마스터가 검토하고 병합할 수 있도록 요청을 보낼 수 있다.

준비 사항

저장소를 만들고 PR 요청할 다른 사용자 추가

우측 Add people 클릭 후 사용자 정보 입력

브랜치 생성 후 코드 수정

$ git checkout -b pr-test

>> 코드 수정(직접 수정해도 상관없음)
$ nano work.txt

$ git commit am "number to english"
$ git push -u origin pr-test

PR 생성

Pull requests 탭에서 Create pull request 버튼 클릭

PR이 없다면 브랜치가 main ← main 일 수 있으니 대상 브랜치를 변경해 주도록 한다.

PR 작성

리뷰 팀원 및 PR의 담당자를 선택하고 현재 PR에 대해 제목과 내용을 작성한다.

• Reviewers : 리뷰해 줄 팀원 선택
• Assignees : PR의 담당자 선택

코드 리뷰

PR 작성 후 Files changed 진입 후 코드에 마우스를 올려 + 버튼 누르면 1줄에 대해 드래그를 하면 여러 줄에 대해 리뷰 및 코멘트 작성이 가능하다.

• Add single comment
  PR에 바로 반영되는 코멘트로 작성 시 바로바로 알림 전송, 다른 팀원 확인이 가능
  매 코멘트마다 알림이 갈 테니 지양하도록 하자.
• Start a review
  모든 리뷰 작성 후에 Review changes를 해야 작성 반영

Add ad Suggestion(ctrl+g)

코드 변경에 대한 의견 제시로 suggestion 내부에 코드 작성 시 PR 대상이 해당 코드로 변경할 수 있도록 유도해준다.

아래는 Suggestion 받은 대상이 볼 수 있는 화면으로 Commit suggestion 클릭 시 해당 의견에 대한 코드가 저장소에 반영이 된다.

코드 리뷰 작성 완료

Review changes를 클릭하면 나오는 화면으로 리뷰에 대한 타입 설정을 할 수 있다.

• Commit : 승인 없는 일반적인 피드백
• Approve : 피드백 제출하고 변경사항 병합 승인
• Request changes : 병합 전 해결해야 하는 피드백 제출

이후에 필요 없는 브랜치는 제거하도록 한다.