git PR 코드리뷰

코드 리뷰는 왜 하는가?

 

               - 내가 이해한 기능에 대해 제대로 구현이 됐는지 

               - 더 나은 코드나 로직이 있는지 

               - 이슈 가능성이 있는 코드는 없는지

               - 확장 가능하고 유연한 코드인지

               - 다른 사람도 알기 쉬운 네이밍을 했는지

               - 개발팀에서 정한 코딩스타일을 가졌는지

               - develop에 연관된 기능변화에 영향도는 없는지 

               - 혼자 판단하기 어렵고 팀원들의 확인이 필요하다

 

 

코드 리뷰 방법은?

 

                 - Git Hub Pull request를 통한 방법

 

리뷰 요청자는 어떻게 요청해야 하나?

 

                - 리뷰어가 쉽게 이해 할 수 있도록 코드에 대한 상세한 설명이 필요함

                - 코드에 대한 확신이 없는 부분을 중점적으로 작성한다

                - 기타 궁금한 점을 적는다

 

코드 리뷰어는 무엇을 중점으로 리뷰할것인지?

 

                   - 이미 구현이 되어 있는 로직이 있는데 중복으로 작성했는지?

                 - 더 효율적인 코드는?

 

PR 생성하기 및 코드 리뷰하기           

1. 브랜치 로컬에 클론 받고 develop으로 checkout 해서 최신 코드를 pull 받는다.
2. 로컬 develop 브랜치에서 feature/지라번호 형식으로 feature 브랜치를 생성한다.
3. 지라를 기반으로 기능 개발건에 대한 내용을 적는다.
4. 특이사항이나 문의사항은 보고자나 기획자에게 문의한다.
5. 지라 내용을 토대로 기능 개발을 한다.
6. 개발이 완료되면 개발자 테스트를 한다
7. 수정된 파일은 커밋하기 전에 그룹핑 해서 커밋 커멘트를 남긴다
8. origin/feature/지라번호 브랜치로 푸쉬를 한다