87일차(Pull request)

Pull request

 

일단 해보고 구글링

지금까지 프로젝트 하면서 한번도 pull request를 해본적이 없었는데 오늘부터 기능을 추가하면 

계속적으로 pull request했다.

 

처음에 아래 버튼 중에 뭘 해야할지 몰라서 첫번째로 먼저 진행했다.

 

 

그랬더니 내가 바로 merge pull request를 진행할 수 있어서

이거는 승인을 안받아도 되는거고 Draff pull request는 승인을 받아야 진행되는 건가 싶었다.

 

구글링 해보니 PR 이슈를 초안(Draft)모드로 오픈할 수 있는 기능을 제공한다고 한다.

Draft 모드인 PR 이슈는 merge 버튼이 활성화되지 않는다.

아직 작업 중이라는 내용으로 코드가 merge될 준비가 아직 되지 않았다는 뜻이다.

 

Ready for review 버튼을 누르면 Draft 모드에서 PR 이슈로 전환되어 코드를 머지할 수 있게 된다.

 

 

 

 

 

문제점 1

이제 2개의 구분 법을 알아서 create pull request로 진행했는데

내가 승인을 할 때 코드를 잘 봐야할 것 같다.

 

처음에 다른 팀원이 승인을 요청했는데 처음이니깐 코드를 확인해봐야지 하고 봤다가

최근에 내가 commit 한 것으로 코드를 합친게 아니라 이 전에 올린 commit으로 pull request하셔서

comment를 등록하고 close했다. (안봤으면 다시 같은 일을 또 할 뻔 했다..)

 

내가 pull request할 때도 팀원분들한테 conflict 안떴다고 했더니 바로 merge하라고 하셨었는데

이렇게 하다가 방금처럼 될까봐 꼼꼼히 봐달라고 했다.

 

 

 

 

문제점 2

다시 팀원분이 승인을 요청해서 코드를 보는데 이해가 안가는 부분이 있어서 review를 달았다.

처음에 Add single comment를 할까 Start a review를 할까 고민되어서 검색을 해보니 

코멘트 or 리뷰+코멘트 차이라고 해서 Start a review로 작성했다.

 

코드리뷰를 작성하고 새로고침하니깐 merge가 되어있었다 (....???)

확인을 해보니 또 다른 팀원이 내가 보고 comment 안달고 넘긴 줄 알고 merge를 했다고 하셨다.   

나는 review를 달아서 띄워져 보이는데 왜 못보셨지? 생각해서 여쭤봤더니 안보이신다고 하셨다.

나는 아래처럼 보여서 캡쳐를 했는데 안보이신다고 하셨다 (...???)

 

 

이것저것 살펴보다가 안건데 코드 리뷰를 하고 아래 버튼인 Review changes를 눌러야

다른 팀원분도 볼 수 있었다.... 😭

 

 

 

 

 

 

해결하기

그래서 이와 같이 코드를 확인안하고 merge하는 경우를 방지하기 위해서 방법이 없을까 하다가

발견한 방법은 아래와 같다. 

 

Review changes를 보면 아래와 같이 Comment, Approve, Request changes가 뜨는데 

이 것을 활용하기로 했다.

 

 

여기서는 리뷰 총평 comment를 달고 토글 버튼을 체크하여 Submit review를 누르면 리뷰가 마무리 된다. 

• Comment : 승인과 무관하게 일반적인 커멘트를 할 때 선택한다.
• Approve : Comment와 다르게 리뷰어가 승인을 하는 것으로, 머지해도 괜찮다는 의견을 보내는 것이다.
• Request changes : 말 그대로 변경을 요청하는 것으로, 승인을 거부하는 것을 뜻한다. 

 

 

 

그리고 프로젝트 세팅에 들어가 보면 브랜치를 관리하는 설정이 있는데,

Approve를 받아야지 merge할 수 있게 설정할 수 있다는 것을 보았다.

 

그래서 2명을 설정해서 2명의 Approve를 받아야지 merge할 수 있게 설정했다.

 

 

 

* 설정에 관해서 해석이 필요하다면 아래 링크를 참고하면 된다.

Git에 대해 알아보자 5. GitHub 추가 기능들

 

 

 

 

출처

 

[Git] GitHub Draft Pull Request 기능

GitHub 사용법 문의합니다. PR review 시의 "Add single comment" 와 "Start a review" 의 차이는 무엇인가요? #47

[Github] Pull request를 통해 코드 리뷰(Code Review)하는 법