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

Feature/#085 auto_merge_approved_pr.yml 오류 수정 #93

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

minjungw00
Copy link
Collaborator

📝 변경 사항

🔍 변경 사항 설명

  • 다음 경우에 대해 에러 대신 상태 메시지가 나오도록 했습니다.
    • approve 수가 부족할 때
    • PR이 이미 닫혀있을 때
    • 병합 충돌이 있을 때

🙏 질문 사항

  • 리뷰어에게 부탁하고싶은 체크리스트를 추가합니다.

📷 스크린샷 (선택)

  • UI 변경이 있는 경우 스크린샷이나 GIF를 첨부합니다.

✅ 작성자 체크리스트

  • Self-review: 코드가 스스로 검토됨
  • Unit tests 추가 또는 수정
  • 로컬에서 모든 기능이 정상 작동함
  • 린터 및 포맷터로 코드 정리됨
  • 의존성 업데이트 확인
  • 문서 업데이트 또는 주석 추가 (필요 시)

@minjungw00 minjungw00 added Bug 기능 또는 UI에서 발생한 오류나 예상치 못한 동작을 해결 Build 개발 환경 세팅(eslint, 컨벤션 등) 또는 패키지 매니저 수정 labels Nov 11, 2024
@minjungw00 minjungw00 self-assigned this Nov 11, 2024
Copy link
Collaborator

@hyonun321 hyonun321 left a comment

Choose a reason for hiding this comment

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

5회 재시도를 할수도있군요. 수고하셨습니다.

@pipisebastian pipisebastian self-requested a review November 12, 2024 01:10
Copy link
Member

@pipisebastian pipisebastian left a comment

Choose a reason for hiding this comment

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

두근..두근..

Copy link
Collaborator

@Ludovico7 Ludovico7 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 확인했습니다!

@github-actions github-actions bot merged commit 98b1c78 into dev Nov 12, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 기능 또는 UI에서 발생한 오류나 예상치 못한 동작을 해결 Build 개발 환경 세팅(eslint, 컨벤션 등) 또는 패키지 매니저 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants