Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[공지사항] PR, Review 남기는 방법, Approve 방법 #11

Open
SeonHyungJo opened this issue May 13, 2019 · 4 comments
Open

[공지사항] PR, Review 남기는 방법, Approve 방법 #11

SeonHyungJo opened this issue May 13, 2019 · 4 comments
Assignees

Comments

@SeonHyungJo
Copy link
Collaborator

SeonHyungJo commented May 13, 2019

PR을 날리는 과정 살펴보기

기존 : fork를 해서 PR을 날리는 방법
현재 : Im-D/Dev-Docs 를 clone 받아서 처리하는 방법



Repo 내려받기

  1. 자신이 원하는 위치로 이동한다.
  2. git clone으로 해당 레포지토리를 내려받는다.
git clone https://github.com/Im-D/Dev-Docs.git

서버에 커밋하기

  1. 자신의 브랜치를 만든다.

브랜치를 만드는 기준은 관리 방법 논의에서 나온 기준으로 만든다.

{ Date }/{ your_name }
Ex) 20190512/SeonHyungJo

  1. 자신의 글을 맞는 폴더 위치에 만든다. (맞는 폴더가 없는 경우 새로 생성한다.)
  2. 글작성이 완료되면 add, commit을 한다.

커밋메시지는 관리 방법 논의에서 나온 기준에 따라 적는다.

:pencil2: {자신의 이름} / {자세한 내용}

ex) :pencil2: SeonHyungJo / Add 최신_브라우저의_내부_살펴보기.md

  1. 서버에 push를 한다.

push를 할 때는 자신의 현재 브랜치를 확인하고 서버의 자신의 브랜치와 동일한 브랜치에 push를 한다.

git push origin { Date }/{ your_name }
// ex) git push origin 20190512/SeonHyungJo

PR작성하기

  1. Im-D/Dev-Docs에 들어가서 새로운 pull request를 만든다.

  2. pull request 작성시 제목은 커밋메시지의 뒤에 있는 내용을 적어준다.

ex) 최신 브라우저의 내부 살펴보기

  1. reviewer는 모든 contrubutor로 설정한다.

1

  1. label은 기본값으로 needs review를 넣는다.

2

  1. ❗️ PR내용에는 자신이 작성한 글의 주소를 넣어주세요 ❗️

9

서버에 push를 하게되면 자신의 브랜치가 생성이 된다. 자신의 브랜치로 변경을 한 후 작성한 글로 이동하여 주소창에 주소를 복붙하면 된다.



Review 방법

3

내가 보고자 하는 PR을 들어가게 되면 상단에 FileChanged를 클릭한다.

4

리뷰를 하고자 하는 line에 마우스를 올려놓게되면 +버튼이 생기게 되고 클릭을 한다.

5

해당 라인에 대한 Comment를 남긴다.

6

우측 상단에 위치한 finish your review를 클릭하여 자신의 리뷰를 등록한다.

7

등록하게 되면 PR의 하위에 생성이 된다. 또한 해당 리뷰에 Comment를 올릴 수 있어 소통이 가능하다.

8

반영이 완료가 되거나 끝난 리뷰는 해결처리를 한다.

Approve 방법

해당 PR에 대해서 나는 승인을 했다라는 의미를 부여한다.

github_approval

File Changed에서 Approve를 선택하고 Comment를 날려주면 된다.

기본은 아래를 따른다.

확인했습니다. 👍 
@SeonHyungJo SeonHyungJo added the 공지사항 공지사항 label May 13, 2019
@SeonHyungJo SeonHyungJo pinned this issue May 13, 2019
@SeonHyungJo SeonHyungJo changed the title [공지사항] PR을 날리는 과정 살펴보기 [공지사항_수정중] PR을 날리는 과정 살펴보기 May 13, 2019
@SeonHyungJo SeonHyungJo changed the title [공지사항_수정중] PR을 날리는 과정 살펴보기 [공지사항] PR을 날리는 과정 살펴보기 May 13, 2019
@SeonHyungJo SeonHyungJo changed the title [공지사항] PR을 날리는 과정 살펴보기 [공지사항] PR, Review 남기는 방법 May 13, 2019
@SeonHyungJo SeonHyungJo assigned Dae-Hwa and ghost May 13, 2019
@SeonHyungJo SeonHyungJo changed the title [공지사항] PR, Review 남기는 방법 [공지사항] PR, Review 남기는 방법, Approve 방법 May 15, 2019
@SeonHyungJo
Copy link
Collaborator Author

SeonHyungJo commented Jul 21, 2019

http://speller.cs.pusan.ac.kr/

글을 작성하는데 도움이 될 만한 사이트를 올립니다.

맞춤법 검사 사이트

@SeonHyungJo SeonHyungJo assigned jigi-kim and JegalEun and unassigned ghost Feb 12, 2020
@SeonHyungJo SeonHyungJo pinned this issue Feb 12, 2020
@cpprhtn
Copy link

cpprhtn commented Nov 22, 2020

Thank you
It helped me a lot.

@dmswl98
Copy link

dmswl98 commented Oct 27, 2022

감사합니다! 참고하겠습니다!

@ensnif060305
Copy link

thankyou

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests