[Git] Pull Request 코드리뷰 규칙 설정, 깃허브 merge 종류

__코드리뷰 받아야 pr merge 되게 깃허브 규칙 만들기 __
Pull Request 머지 종류 정리, 코드리뷰 하는 법 __

코드리뷰 풀리퀘 머지 전 승인규칙 설정하기

레포지토리에 풀리퀘스트 후

참여자들의 코드리뷰 승인approve 받아야지만 merge 될 수 있도록 하기

1. repostitory 설정 들어가기
   
   
2. Settings > Branches > Add branch rulset
   
   

• 규칙이름, 활성화여부 선택
  
  
• 어떤 브랜치에 적용할지 고르기
  (패턴으로 정할수도 있음)
  
  
• merge 전 풀리퀘, 승인 필요함 체크
  여기서 인원수는 본인 제외
  
  
• 최소 1명 리뷰 필요하다고 머지 막힘
  
  

코드리뷰하는법 approve

add review 또는 커밋 코드들 들어가기

1. 코드의 원하는 부분에 댓글달기
   
   
2. 파일에 코멘트
   
   
3. 리뮤 완료하기
   
   
4. 최종 멘트 작성
   
   
   
   comment - 멘트만
   approve - 승인 (이거해야 머지가능)
   requst changes - 변경사항 요청

머지하기

 

머지 규칙?

1. 기존 딸려온 커밋 기록 + 머지했다는 커밋
2. 기존 딸려온 커밋들 다 합쳐서 머지 커밋에 넣고 이거 하나만 커밋
3. 기존 딸려온 커밋들 기존거 뒤에 그냥 일자로 붙이기

예제
기존 a 라는 커밋이 있었고
풀리퀘에 new a1 new a2 이라는 커밋이 딸려있을 경우

Create a merge commit

다른 branch 파생을 나타내는 그래프가 그려지며, merge 커밋이 추가됨

|
Merge pull request #3 from keartt/main
|
|\
| * new a2
| * new a1
|/
|
a

Squash and merge

합쳐서 새로운 커밋 new a 를 만듦

|
new a
|
a

Rebase and merge

그냥 현재 레포에서 작업한것처럼 일자로 생성

|
new a2
|
new a1
|
a