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

Label 정리하기 #638

Open
hg-pyun opened this issue May 14, 2023 · 3 comments
Open

Label 정리하기 #638

hg-pyun opened this issue May 14, 2023 · 3 comments
Assignees
Labels
discussion Discuss the problem.

Comments

@hg-pyun
Copy link
Member

hg-pyun commented May 14, 2023

현재 PR에서 사용하고 있는 Label 중에 유의미하지 않은 것들을 정리해보면 좋을 것 같습니다

  • beta: 더이상 사용하지 않으므로 제거
  • needs review: PR자체가 리뷰가 필요하므로 중복
  • needs author response: 이것 역시 comment, change request, approved가 같은 의미를 나타내므로 제거
  • ready to merge: approved와 중복

그밖에 새로 추가되면 좋을 법한 Label 이 있다면 의견주시면 감사하겠습니다. 🙇‍♂️

@hg-pyun hg-pyun added the discussion Discuss the problem. label May 14, 2023
@hg-pyun hg-pyun self-assigned this May 14, 2023
@taggon
Copy link
Collaborator

taggon commented May 23, 2023

needs author response는 "이미 누군가 검토했고 PR 작성자의 응답을 기다리는 중이다"라는 정보를 줄 수 있다고 생각합니다. 다른 리뷰어들의 시간을 절약해 줄 수 있을테고 혹시나 코멘트 내용을 잘못 보고 승인하는 경우도 방지할 수 있을 거 같습니다. Change Request로 하기에는 조금 애매한데 의견이 필요한 경우도 있지 않을까요?

나머지 레이블은 없애는 데 동의합니다.

@hg-pyun
Copy link
Member Author

hg-pyun commented May 29, 2023

@taggon 넵 그럼, needs author response 태그만 남겨두고 일단 나머지는 제거해보도록 하겠습니다. 🙇‍♂️

@eomttt
Copy link
Collaborator

eomttt commented Jun 5, 2023

번역자가 번역 완료 해서 리뷰어가 다시 확인해야하는 경우도 필요할것 같은데 일단 ready to merge 를 붙여놨습니다
확인후 머지 하면 될것 같아요

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

No branches or pull requests

3 participants