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

PR 코드 분석 시 suggestion 태그를 위한 로직 추가 #765

Merged
merged 2 commits into from
Dec 10, 2023

Conversation

JJ503
Copy link
Member

@JJ503 JJ503 commented Dec 8, 2023

📄 작업 내용 요약

PR 코드 분석 시 suggestion 태그를 위한 로직 추가

🙋🏻 리뷰 시 주의 깊게 확인해야 하는 코드

지난 제안 pr에서 추가된 사항은 아래와 같습니다.

  • 제안 pr workflow는 무조건 실패하도록 함
  • 리뷰어는 @channel을 통해 호출
    -> 이는 추후 프로젝트에 더 이상 참여하지 않을 경우 채널에서 남가는 방식이 가장 편할 것이라 판단했기 때문입니다.

📎 Issue 번호

@JJ503 JJ503 added environment 환경설정 변경 시 backend 백엔드와 관련된 이슈나 PR에 사용 labels Dec 8, 2023
@JJ503 JJ503 requested review from apptie, swonny and kwonyj1022 December 8, 2023 07:08
@JJ503 JJ503 self-assigned this Dec 8, 2023
Copy link
Collaborator

@swonny swonny 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

@kwonyj1022 kwonyj1022 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@JJ503 JJ503 merged commit bc5dd6a into develop-be Dec 10, 2023
2 checks passed
@JJ503 JJ503 deleted the env/740 branch December 10, 2023 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend 백엔드와 관련된 이슈나 PR에 사용 environment 환경설정 변경 시
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants