[생활코딩_GIT] 07. git hub에서 코드 리뷰하기 (feat. pull request)

# pull requset란? 

- 내가 한 작업을 끝냈을때, 다른사람들의 코드리뷰(조언)을 받고싶을 때 사용하는 기능

- 내가 작업한 코드를 다른멤버들이 검토해서, 코드의 품질을 높이고, 통합 브랜치의 안정성을 높이는 것이 핵심.

 

- 프로젝트에 대입해서 생각하면..

보통 많은 프로젝트들에서 master branch는 바로 배포해도 문제가 없을 정도로 무결한 상태를 유지해야한다.

그래서 작업시에는 작업용 branch를 따로 만들어 두어 사용하는 것을 권장한다. 그리고 이 branch를 병합 하기 전, 코드가 무결한지 검사를 받는 과정이다.

 

*참고 : git hub에서 pull request  = git lab의 merge request 와 같은 말이다.

 

 

# pull request 의 종류

1. 원격저장소가 나의 것이거나, 내가 원격저장소에 직접적으로 접근할 수 있는 경우 (이번에 다룸)

- 내가 작업한 브랜치를 다른 사람들의 브랜치와 병합 할 때, 다른사람과의 토론을 받고싶을때 쓴다.

 

2. 원격저장소에 대한 접근권한이 없고, 원격저장소를 복제후(fork), 작업하고 원격저장소를 개선시켜 나가는 것.

(오픈소스에서 자주사용)

 

 

 

# git hub 에서 pull-reqest 하기 01

1. 내 repository 에서 pull-request 버튼 클릭

2. 작업이름 , 내용 , review를 해줬음 좋겠는 사람 등을 선택하고 올린다.

3. (최종 컨펌자의 경우) 올라온 pull request 가 문제가 없다면, [merge pull request]버튼을 눌러주면된다.

 

4. [confirm merge]클릭

 

 

5. 그럼 master branch에 해당 branch가 병합된다.

6. git 에서 확인하면..

- git pull 후 git graph에서 확인해 보면, 병합이 잘 되었음을 확인 할 수있다.

 

 

 

 

# git hub 에서 pull-reqest 하기 02

1. 상단 pull request 탭에서 [ new pull request ] 버튼 클릭

2. pull-request 하고자 하는 branch 를 선택한다.

3. [ create pull request ] 버튼 클릭

3. 작성 후 초록색 버튼 클릭 시 pull-request가 올라간다.

 

 

# 다른사람의 pull-request review해주기

1. 원하는 수정 라인을 drag 후, + 버튼을 클릭하면, 아래와같이 댓글창이 나온다. 

내용을 입력후, [ add single comment ] 클릭시에는 해당 라인에 대한 리뷰가 바로 달리고,

[ start a review ] 클릭 시, 떨어져있는 다른 라인에도 함께 코드에 대한 리뷰를 달 수있다.

2. 코드에대한 리뷰를 완료했다면 오른쪽 상단 초록색 아이콘 클릭,

댓글 남기고, 하단 초록색 버튼 클릭하기.

 

클릭시 나오는 옵션들은 다 강제력이 없고, 비슷한 의미이긴한데, 늬양스를 약간 설명해보자면

1) comment : 그냥 댓글달기

2) Approve : 머지에 동의했단 의미

3) Request changes : 조금 더 강력하게 변경햇슴.. 하는것