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

workflows size-label #14

Merged
merged 1 commit into from
Feb 21, 2023
Merged

workflows size-label #14

merged 1 commit into from
Feb 21, 2023

Conversation

jmnote
Copy link
Contributor

@jmnote jmnote commented Feb 20, 2023

What this PR does / why we need it (변경 내용 / 필요성):

코드리뷰의 분량을 쉽게 알 수 있도록
PR의 사이즈에 따라 label을 붙여주는 github actions을 추가하고자 합니다.

별도 repo에서 테스트하여 이상없음을 확인하였습니다.
소스코드: https://github.com/kuoss/goproject1/blob/main/.github/workflows/size-label.yml
테스트결과: https://github.com/kuoss/goproject1/pulls

Which issue(s) this PR fixes (관련 이슈):

없음

Special notes for your reviewer (리뷰어에게 하고 싶은 말):

리뷰 감사합니다. 파이팅!

Additional documentation, usage docs, etc. (기타 관련 문서, 사용법 등):

공식문서(repo) https://github.com/pascalgn/size-label-action

단, 공식문서의 사용법 내용이, 현재는 제대로 작동하지 않아서 퍼미션을 추가하였고
해당 repo에도 문서를 수정하도록 PR을 등록했습니다.
1월 12일에 PR을 등록했는데, 2월 20일 현재까지 리뷰는 되지 않고 있네요.
pascalgn/size-label-action#34

참고로 사이즈별 레이블명은 다음과 같습니다. ( 00줄 이상인 경우 레이블 xx )

{
  "0": "XS",
  "10": "S",
  "30": "M",
  "100": "L",
  "500": "XL",
  "1000": "XXL"
}

@jmnote jmnote added the documentation Improvements or additions to documentation label Feb 20, 2023
@jmnote jmnote self-assigned this Feb 20, 2023
Copy link
Contributor

@dozer-jang dozer-jang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다~ 0줄인 경우, XS 라벨이 붙는데 혹시 어떤 경우가 있을까요?

@jmnote
Copy link
Contributor Author

jmnote commented Feb 21, 2023

단순히 정수의 영역을 나누다 보니 그런 것이고, 실제로 0줄인 경우는 없지 않을까요?

원전(?)을 따라가보니 아래 링크가 사실상의 표준이라고 할 수 있겠고요.
https://github.com/kubernetes/kubernetes/labels?q=size

실제 size/XS 중에 0줄인 게 있는지 찾는 건 어렵겠네요. ㅎㅎ

@jmnote
Copy link
Contributor Author

jmnote commented Feb 21, 2023

남은 의문점이 없으면 merge 하고자 합니다. ㅎㅎ

@jmnote
Copy link
Contributor Author

jmnote commented Feb 21, 2023

확인(👍) 감사합니다. merge 합니다.

@jmnote jmnote merged commit 77f866b into main Feb 21, 2023
@jmnote jmnote deleted the workflows-size-label branch February 21, 2023 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants