커밋 합치는 걸 허락해 줘! pull request
fork한 저장소에서 기능 개발을 마치고, 해당 코드를 원본 저장소에 merge해야 한다.
권한이 없다면 pull request를 보낸다.
- 머지하고 싶은 두 브랜치를 선택한다.
- 어떤 변경을 했는지 제목과 내용을 작성한다.
- 단일 저장소에서 보낼 수도 있고, 포크한 저장소에서 보낼 수도 있다.
pull request로 merge 요청 보내기
- 여러 개발자와 협업중이라면, 최대한 직접 merge는 피하고 모든 merge를 pull request를 통해서 진행한다.
- 동료 개발자가 내 PR을 보고 코드를 리뷰할 수 있다.
- 동료의 PR에 수정이 필요하다면 댓글을 달아 change request를 보낼 수 있다. (ex 이코드는 수정이 필요합니다. PM에게 듣기로는 timeout을 10초로 해야 합니다.)
- 오픈소스에 PR을 보낼때는 contribution guideline을 반드시 참고해야 한다.
Able to merge라고 보여지면 comflict없이 merge 할 수 있다.
conflict가 난다면 미리 내 저장소에서 해결한 다음 PR을 보내야 한다.
Tip
-> 내용에 내가 수정한 코드 스크린샷으로 첨부하면 상대방이 코드리뷰하기 쉽다.
-> 원본 저장소에서 File changed 탭을 클릭하면 review를 달 수 있다.
-> Conversation 탭에서 리뷰 확인 가능
GitHub에서 PR확인하고 merge하고 로컬 저장소로 pull 하면 완료