6. 다른 사람의 Pull request 코드 리뷰하기
Pull request에 들어가면 여러 개의 탭이 있는데, 그 중 Files changed를 클릭하면 어느 파일이 어떻게 변경되었는지 모두 확인이 가능하다.
우선 직접 테스트해보지는 못하지만 코드를 읽으면서 수정이 필요한 line number에 마우스 커서를 대면 파란색으로 + 버튼이 생긴다.
클릭해서 코멘트를 남기고 Start a review 버튼을 클릭한다.
모든 리뷰가 끝나면 우측 상단에 Finish your review 버튼을 클릭하고
Comment에 체크한 다음 Submit review 버튼을 클릭한다.
상황이 변경되어 라벨 변경이 필요하다면 라벨도 바꿔준다.
더이상 수정이 필요한 부분이 없다면, Pull request의 Files changed 부분에 들어가서 우측 상단 Review changes 버튼을 클릭한다.
코멘트 내용으로는 고생했다는 칭찬을 아낌없이 해주고, 두 번째 Approve에 체크한 다음 Submit review 버튼을 클릭한다.
마찬가지로 상황이 변경되어 라벨 변경이 필요하다면 라벨도 바꿔준다.
💡 PR 올라온 코드 로컬에서 실행시켜보는 방법
- branch 새로 만들고 이동
$ git checkout -b [branch name]
- PR올린 사람의 브랜치 pull 받기
$ git pull [팀원 repository HTTPS] [팀원이 작업한 branch name]
ex) 주현이가 combo_bug branch에서 작업한 내용을 PR했을 때
$ git pull https://github.com/hyeoneedyou/2021-1-OSSPC-WelcomeOS-1.git combo_bug
최신코드 받을 때와 마찬가지로, merge를 하면서 생기는 commit에 대한 부분인데, 따로 우리가 무언가를 해 줄 필요는 없다.
Ctrl+X(Esc)를 누르고 :wq를 입력한 다음 엔터를 누른다.
- 실행해보기
$ cd BlockKing
$ python3 blockking.py
- 확인이 완료되면 해당 branch 삭제하기 다른 branch로 이동해야 삭제할 수 있다.
$ git checkout [기존에 있던 branch name]
$ git branch -d [이번에 만든 branch name]
merge가 안 돼서 정말 삭제할거냐는 문구가 뜨면 그대로 진행
$ git branch -D [이번에 만든 branch name]