-
-
Notifications
You must be signed in to change notification settings - Fork 457
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
제안) 병합에 필요한 Require Approvals를 2로 변경 #625
Labels
Comments
#374 (comment) 에서 Dan이 언급했듯이 현재 모든 메인터넌서가 액티브하게 참여하고 있지 않은 상황이긴합니다 흑흑.. 2명이상 리뷰가 잘 될지 혹은 속도를 늦추지는 않을지 걱정이 약간 되네요. 추가로 번역외에 기여하고자 하는 분들을 유도하기 위한 방법 중 하나로는 좋은 방법인 것 같습니다. 다만 이게 얼마나 효용성이 있을지는 미지수군요. 일단 조금 더 탄탄한 리뷰가 가능할 것 같고, 머지를 위해 다른 사람들을 독촉도 해야하니(?) 2개로 바꿔보고 상황을 살펴봐도 좋을 것 같다는 생각입니다. |
큰 이견 없습니다 ㅎㅎ |
생각보다 2 approved를 받는게 쉽지 않아서, 당분간은 1로 운영하면 어떨까 싶습니다. |
@gnujoow 1로 바꿔뒀습니다 🙇♂️ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
지금 Pull request를 병합하기 위한 조건으로 하나 이상의 approve를 필요조건으로 설정해두었습니다. #374 의 상단에서 확인할 수 있듯이 10분의 메인테이너 분들과 함께하고 있고, 새로운 문서로 번역이 진행되는 점을 고려했을때, 병합에 필요한 approve를 2로 변경을 제안합니다.
추가로 #374 에 대부분의 문서들이 번역이 할당 되었는데, 추가로 참여를 원하시는 분들은 문서 번역 검수를 같이 진행하는것도 어떤지 제안 해봅니다.
The text was updated successfully, but these errors were encountered: