코드리뷰와 PR 종류 선택에 대해 강사님의 생각이 궁금합니다.
402
작성한 질문수 13
코드 리뷰를 작성하고 제출할 때(07:16) Comment, Approve, Request Changes 가 있습니다.
PR 후 merge를 할 때도 Merge commit, Squash and merge, Rebase and merge 가 있습니다.
각각에 대해서 어떤 상황에 어떤걸 선택하시는지 강사님의 경험(?)과 생각이 궁금합니다 ㅎㅎㅎ
강의 잘 보고 있어요! 감사합니다!
답변 1
0
코드리뷰 제출시)
초반에 서로 코딩 스타일을 볼 때는 버그가 날 수 있는 코드 파악 뿐 아니라 소소한 코딩 컨벤션 / 아키텍쳐 논의까지 모두 리뷰 커멘트에 달고 'Comment'를 남기는 편이고요(오픈소스처럼 빡세게 해야하는거 아니면 보통 Request Change는 쓰지 않아요)
어느정도 서로 개발 스타일 알게 되면 소소한 커멘트는 스킵하는 편입니다 ㅎㅎ (이것도 Comment)
좋다면 바로 LGTM이란 커멘트와 함께 Approve합니다.
Merge시)
Squash and merge를 선호합니다~ admin계정은 GitHub 레포 세팅에서 원하는 머지방식만 남겨놓을수있는데 제가 관리하는 레포엔 Squash방법만 남기는 편이에요.
커밋이 PR단위로 하나로 퉁쳐져서 팀단위로 보기 편한게 이유입니다~
pull할때
0
52
1
강의자료 부탁드립니다.
0
60
1
강의자료 부탁드립니다.
0
50
1
강의 자료 부탁드립니다.
0
60
1
강의 자료 부탁드립니다.
0
57
1
06:48 원래한번에 포크 안받아져야 정상인가요?
0
59
1
저는 풀버튼에 풀받을게 없다고 뜹니다.
0
52
1
5:10부터 저는 모든 태그 푸시가 자동체크 안됐는데..
0
48
0
git remote add origin ~~ 질문요
0
78
1
강의자료 부탁드립니다
0
47
1
강의자료 요청드립니다.
0
63
1
github 메뉴 항목 차이점
0
142
1
강의 자료 부탁 드립니다.
0
123
2
안녕하세요! 강의 자료 부탁드립니다!
0
104
1
강의자료 부탁드립니다!!
0
94
1
강의 ppt 자료 부탁드립니다.
0
120
1
강의자료 부탁 드립니다.
0
133
1
강의자료 부탁드립니다.
0
107
1
강의자료 부탁드립니다
0
99
1
강의 자료 부탁드립니다.
0
198
2
강의자료 부탁드립니다
0
144
2
강의자료부탁드립니다!
0
141
2
강의 자료 부탁드립니다.
0
153
2
강의 자료 부탁드립니다.
0
181
2