Skip to content

Latest commit

 

History

History
77 lines (42 loc) · 2.44 KB

PULL_REQUESTS_GUIDE.md

File metadata and controls

77 lines (42 loc) · 2.44 KB

PULL_REQUESTS_GUIDE


Pull Request
PR 수정
Approve


Pull Request

1. 본인계정 fork 한 레퍼지토리에 문제 commit

  • 본인이 fork한 레퍼지토리를 clone 했기 때문에, 본인 저장소에만 push가 된다.

Untitled (36)

2. Pull requests > New pull request

Untitled (37)

3. 작성 > Reviewers 설정 > Create pull request

  • 본인 브랜치에서 공용 레퍼지의 main 브랜치로
  • 제목 양식 : [이름 / 0월 0주차/요일]
  • 내용은 자유, 적지 않아도 무관하다.
  • (선택) 리뷰어는 페어를 태그한다.

Untitled (38)

4. PR 완료

  • 공용 저장소에서 확인 할 수 있다. Untitled (39)

5. 클릭하면 PR 내역 확인 가능

Untitled (40)


PR 수정

6. PR 수정

  • 본인의 PR이 Open 이라면, 본인의 repository에 수정한 것만으로도 자동 수정이 된다.
  • 즉, 별도의 작업 없이 본인이 fork한 저장소에 push만 하면 끝!

Approve

7. 페어와 코드리뷰를 진행 완료한 경우,

해당 PR > Files Changed > Review changes > Approve에 체크하고, Submit review 해준다.

  • 2명 이상 Approve 해 주어야, Merge가 가능하다.

8. Approve 되지 않아 경고가 뜬 화면

Untitled (41)

9. Approve 방법

  • 내용은 작성하지 않아도 무관하다.
  • (+참고) 코드 위에 마우스를 얹으면 + 버튼이 생기고, 이 버튼을 드래그하여 코드를 선택하여 Comment를 달 수 있다. 또한, Request Changes를 체크하게 되면, 수정을 요청해달라는 뜻으로, 수정이 완료될 때 까지 Merge가 불가능하다. Untitled (42)