아하

생활

생활꿀팁

당당한말벌206

당당한말벌206

코드리뷰 시 Pull Request 크기

보통 코드 리뷰시 1회 PR의 크기를 어떻게 사용하는게 좋나요? 너무 크면 변경 내용 확인이 어려워서 시간이 많이 소요됩니다. 변경 내용이 많을때 어떤 기준으로 PR을 나눠서 작성해야하는지 조언 부탁드립니다.

    2개의 답변이 있어요!

    • 탈퇴한 사용자

      탈퇴한 사용자

      안녕하세요.

      PR 기준은 별도로 없는 것으로 알고 있습니다.

      제 생각에는 기능 (feature)이나 버그 수정 (hotfix) 단위로 하는게 좋을 듯 합니다.

      하나의 기능을 개발 또는 버그 수정 시 branch 하나를 생성하고 그 branch를 PR 하고나서 merge가 완료되면

      해당 branch를 삭제하면 좋을 것 같습니다.

    • 검은밀잠자리283

      검은밀잠자리283

      이슈 단위로 Pull Request를 만들어야하지 않을까 합니다.

      1가지 이슈가 너무 크더라도 해당 이슈를 해결하는 코드라면 다 포함이 되야할 것 같습니다.

      회사마다 방법이 다르겠지만 일하는 회사에서는 이슈 생성시 작은 단위로 만드려고 합니다.

      예를 들면 회원 리스트 조회 같은 API 개발을 1개의 이슈로 등록하고 개발합니다.

      버그의 경우 회원조회시 XXX버그 수정 등 단위로 만들어서 하고 있습니다.