인프런 커뮤니티 질문&답변

포닥님의 프로필 이미지

작성한 질문수

팀 개발을 위한 Git, GitHub 입문

풀 리퀘스트(Pull request): 내 코드를 머지해주면 안되겠니?

코드리뷰와 PR 종류 선택에 대해 강사님의 생각이 궁금합니다.

21.09.17 11:15 작성

·

317

0

코드 리뷰를 작성하고 제출할 때(07:16) Comment, Approve, Request Changes 가 있습니다.

 

PR 후 merge를 할 때도 Merge commit, Squash and merge, Rebase and merge 가 있습니다.

 

각각에 대해서 어떤 상황에 어떤걸 선택하시는지 강사님의 경험(?)과 생각이 궁금합니다 ㅎㅎㅎ

 

강의 잘 보고 있어요! 감사합니다!

답변 1

0

진유림님의 프로필 이미지
진유림
지식공유자

2021. 10. 23. 22:54

코드리뷰 제출시)

초반에 서로 코딩 스타일을 볼 때는 버그가 날 수 있는 코드 파악 뿐 아니라 소소한 코딩 컨벤션 / 아키텍쳐 논의까지 모두 리뷰 커멘트에 달고 'Comment'를 남기는 편이고요(오픈소스처럼 빡세게 해야하는거 아니면 보통 Request Change는 쓰지 않아요)

어느정도 서로 개발 스타일 알게 되면 소소한 커멘트는 스킵하는 편입니다 ㅎㅎ (이것도 Comment)

좋다면 바로 LGTM이란 커멘트와 함께 Approve합니다.

 

Merge시)

Squash and merge를 선호합니다~ admin계정은 GitHub 레포 세팅에서 원하는 머지방식만 남겨놓을수있는데 제가 관리하는 레포엔 Squash방법만 남기는 편이에요.

커밋이 PR단위로 하나로 퉁쳐져서 팀단위로 보기 편한게 이유입니다~

포닥님의 프로필 이미지
포닥
질문자

2021. 10. 23. 23:54

제가 사용해왔던 방식과 비슷해서 다행(?)이네요.

감사합니다 !!

포닥님의 프로필 이미지

작성한 질문수

질문하기