[Git] Pull Request의 Code Review 를 필수로 만드는 방법 (commit 시 require a pull request before merging 에러 해결법)

Code Review 를 필수로 하기로 했는데, 제한을 안걸어놨더니 구성원이 실수로 merge 를 하는 일이 발생하지 않는가? 이런 상황을 방지하기 위해 Branch Rule 을 설정한다. 

Branch rule 을 세팅하기 위해 우선, Settings > Branches > Add rule 로 이동한다.

 

 

Branch name pattern

처음에 잘모르고 Branch name pattern을 *(전체) 로 했다가 이클립스를 사용하는 구성원한테 require a pull request before merging 에러가 발생했다. 인텔리제이는 멀쩡해서 처음에 원인 파악이 오래걸렸다. 

와일드 카드(*/전체) 로 Branch name pattern 을 거니 모든 브랜치에 제약이 걸려서 commit 하기 전에 pr을 하세요 에러가 생기는 거였다. 해당 규칙은 main 브랜치에만 걸어도 된다. 

 

Require a pull request before merging 

머지 전 pull request 가 필요하도록 설정하는 것이다. 이걸 체크하고 Require approvals 까지 체크하면 된다.

Required number of approvals before merging 에서 merge 하기 위한 승인 수를 선택하면 된다. 우리 프로젝트의 멤버는 3명이어서, 나를 제외한 두명의 code review 승인을 받으면 되도록 설정해놨다.

 

 

 

 

깃 Issue, Pul Request 템플릿 설정하는 법 보러가기

Github 사이드 프로젝트 Issue, Pull Request 템플릿 설정하는 방법