코드리뷰 2단계는 pull request를 통해 코드 리뷰 요청을 한 후 피드백을 받고, 피드백을 반영하는 과정을 다룬다.
- github 서비스에서 pull request를 보낸다.
pull request는 github에서 제공하는 기능으로 코드리뷰 요청을 보낼 때 사용한다.
pull request는 original 저장소(next-step의 저장소)의 브랜치(자신의 github 아이디)와 앞 단계에서 생성한 브랜치 이름(앞 단계의 예에서는 step1)을 기준으로 한다.
ex) 미션을 진행한 javajigi/java-racingcar step1 브랜치 => next-step/java-racingcar javajigi 브랜치로 pull request를 보낸다.
- 브라우저에서 github 저장소에 접근한다.
- 브랜치를 작업 브랜치로 변경한다(앞 단계의 예에서는 step1).
- 브랜치 오른쪽에 있는 "New pull request" 버튼을 클릭한다.
- 왼쪽 next-step 저장소의 브랜치를 자신의 github 계정 브랜치로 변경한다.
- 현재 미션에서 작업한 내용을 입력하고 "Create pull request" 버튼을 클릭해 pull request를 보낸다.
- pull request를 보냈을 때의 상태는 다음과 같다.
- pull request를 보낸 후 리뷰어에게 리뷰 요청을 한다.
- NextStep(https://edu.nextstep.camp) 에 접속해 자신이 수강하고 있는 강의로 이동한다.
- 우측 상단의 github 로고를 클릭하면 자신에게 배정된 리뷰어에게 리뷰 요청 알림이 간다.
- 리뷰 요청이 정상적으로 동작하면 리뷰어에게 slack으로 알림 요청이 간다.
- pull request에 대해 승인이 되지 않고 수정 요청 피드백을 받으면 피드백 받은 내용을 반영한다. 만약, pull request가 승인이 되어 next-step 저장소에 통합(merge)이 된다면 코드리뷰 요청 3단계를 진행한다.
- 피드백을 반영한 후 add, commit, push 명령을 실행한다.
pull request를 보내 피드백을 받은 후 add, commit, push를 한 후 새로운 pull request를 보내지 않아도 된다.
앞서 보낸 pull request가 통합(merge)되지 않은 상태이기 때문에 같은 pull request를 재활용한다.
git status // 변경된 파일 확인
git add -A(또는 .) // 변경된 전체 파일을 한번에 반영
git commit -m "메시지" // 작업한 내용을 메시지에 기록
git push origin 브랜치이름
ex) git push origin step1
- 새롭게 add, commit을 진행하는 경우 새로운 버전이 생성되어 상태는 다음과 같다.
- 새로운 버전을 push한 상태는 다음과 같다.
- 몇 번의 피드백을 주고 받은 후 승인이 되어 next-step 저장소에 통합(merge)이 된다면 코드리뷰 요청 3단계를 진행한다.
- pull request가 승인이 되어 통합이 된 후의 상태는 다음과 같다.
