21.09.17 11:15 작성
·
304
답변 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
제가 사용해왔던 방식과 비슷해서 다행(?)이네요.
감사합니다 !!