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.

Sign up for GitHub

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

Jump to bottom

📢 Code merge 관련 건 #10

Open

cyeji opened this issue May 7, 2023 · 1 comment

Assignees

Labels

Contributor

cyeji commented May 7, 2023

프로젝트 PR Merge 관련 건 수정사항 업데이트 했습니다.

Before
일괄 merge

Now
Reviewer 신청자가 모두 승인을 했을시에만 merge가능

cyeji added the 📢 Notice label

cyeji self-assigned this

Contributor Author

cyeji commented May 7, 2023 •

edited

Loading

PR 후 Reviewer 등록 방법

Create Pull Request 화면 오른쪽 Reviewers 설정 및 Assignees 설정 (두명 전부 선택할 것)

선택 완료 시 표기되는 화면

Review후 상태 변경

ReviewChanges 버튼 클릭 후, 상태변경 (comment, approve, request changes)

comment (코멘트만 달고 머지 되도록)
approve (승인)
requestChanges (피드백 받고 수정하기 전까지 merge 금지)
상태변경 설정은 유동적으로 설정합니다.

ex. 크게 문제는 되지 않지만 고치면 좋음 Comment, 코드 변경 요청 필요 없는 상태 Approve, 코드에서 변경되지 않으면 문제가 있을 때 requestChanges

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment