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.

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

feat: approved 상태에 따른 PR의 라벨 추가 #20

Merged
merged 2 commits into from
Jul 9, 2024
Merged

feat: approved 상태에 따른 PR의 라벨 추가 #20

merged 2 commits into from
Jul 9, 2024

Conversation

hjy0951
Copy link
Collaborator

@hjy0951 hjy0951 commented Jul 9, 2024

🤔 어떤 문제가 발생했나요?

  • PR의 리뷰 진행 상태를 보기 위해서 항상 PR을 클릭해서 확인해보아야 한다.

🎉 어떻게 해결했나요?

  • GitHub Actions의 workflow에서 PR의 Approved 수를 카운트하여 라벨링을 통해 바로 확인할 수 있도록 하였습니다.

📷 이미지 첨부 (Option)

  • NA

⚠️ 유의할 점! (Option)

  • NA

@hjy0951 hjy0951 added the Review Plz🙏 # Review is not yet complete label Jul 9, 2024
@hjy0951 hjy0951 marked this pull request as ready for review July 9, 2024 12:33
Jungjjeong
Jungjjeong approved these changes Jul 9, 2024
View reviewed changes
Copy link
Member

@Jungjjeong Jungjjeong left a comment

Choose a reason for hiding this comment

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

👍

@hjy0951 hjy0951 added Approved 🆗 # Review is completed and removed Review Plz🙏 # Review is not yet complete labels Jul 9, 2024
@Jungjjeong
Copy link
Member

이야 잘붙네요

@hjy0951 hjy0951 merged commit d8ed00d into main Jul 9, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jungjjeong Jungjjeong Jungjjeong approved these changes

@wokbjso wokbjso Awaiting requested review from wokbjso wokbjso is a code owner

@summermong summermong Awaiting requested review from summermong summermong is a code owner

Assignees

@hjy0951 hjy0951

Labels
Approved 🆗 # Review is completed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hjy0951 @Jungjjeong