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에 대해서 #8

Open
hyena0608 opened this issue Mar 2, 2023 · 0 comments
Open

💡[공지사항] PR, Review, Approve에 대해서 #8

hyena0608 opened this issue Mar 2, 2023 · 0 comments

Comments

@hyena0608
Copy link
Member

PR 요약

  1. 레포지터리를 fork한다.
  2. 아이템 번호로 branch를 만든다. (ex. item32)
  3. 발표 자료를 업로드한다.
  4. PR을 main에 날린다. (main <- item32)
  5. Assignees와 Reviewer를 등록한다.
  6. 2명 이상 approve 해주면 해당 PR을 merge 한다.

Review 요약

  1. 크루원의 PR에 들어가 리뷰를 시작한다.
  2. 수정이 필요 없다고 판단되면 approve한다.

Pull Request 하기

  • 2023-effective-java-study 레포지터리를 fork한다.

  • fork한 레포지터리를 clone한다.

  • 맡은 아이템 번호로 브랜치를 생성한다.

    • ex) 아이템32 -> item32

image




  • 발표 자료를 올리고 PR을 날린다.

image




  • Reviewers에 스터디 크루를 모두 넣는다.

image

image

image




  • Assignees에 자신을 넣는다.

image




  • 해당 PR로 들어가면 다음과 같이 Reviewer가 두 명 이상 approve 되어야 merge 할 수 있다.

image




Review

  • 리뷰어는 File Changes에 들어가서 커멘트를 작성한다.

image




  • 커멘트를 모두 작성하고 나면 Review changes를 눌러 Comment를 작성한다.

image




  • 작성한 리뷰를 볼 수 있다.

image




  • 더 이상 수정할 부분이 없다 판단되면 approve 한다.

image




  • approve 하면 다음과 같이 1명이 approve 했다고 알 수 있다.

image

@hyena0608 hyena0608 pinned this issue Mar 2, 2023
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

1 participant