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

[BE] 코드 리뷰 Github action 연동 #104

Merged
merged 1 commit into from
Jul 24, 2024
Merged

[BE] 코드 리뷰 Github action 연동 #104

merged 1 commit into from
Jul 24, 2024

Conversation

koust6u
Copy link
Member

@koust6u koust6u commented Jul 24, 2024

연관된 이슈

구현한 기능

slack 메세지로 PR에 대한 코멘트가 도착했음을 알림

상세 설명

@@ -0,0 +1,50 @@
name: Notify Slack on Pull Request Review Request
Copy link
Contributor

Choose a reason for hiding this comment

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

!!!

Copy link

Unit Test Results

0 tests   0 ✔️  0s ⏱️
0 suites  0 💤
0 files    0

Results for commit 6ad3544.

Copy link
Contributor

@JiHyeonL JiHyeonL left a comment

Choose a reason for hiding this comment

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

고쳐주세요

Comment on lines +6 to +14
env:
koust6u: U07D1PWSLSG
kelly6bf: U07DSB12K99
JiHyeonL: U07E3T9PMA4
yechop: U07E3TG6CHE
reddevilmidzy: U07DF0QPUKV
dle234: U07E48NCK4G
greetings1012: U07DPGSCXRA
anttiey: U07DPGSCXRA
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분이 아쉬워요

name: Notify Slack on Pull Request Review Request

on:
pull_request_review:
Copy link
Member

Choose a reason for hiding this comment

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

1

with:
channel-id: 'C07DN3TETSL'
payload: |
{
Copy link
Member

Choose a reason for hiding this comment

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

2

Copy link
Member

@reddevilmidzy reddevilmidzy left a comment

Choose a reason for hiding this comment

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

coment

pull_request_review:

env:
koust6u: U07D1PWSLSG
Copy link
Member

Choose a reason for hiding this comment

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

dd

Copy link
Contributor

@JiHyeonL JiHyeonL left a comment

Choose a reason for hiding this comment

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

굿

@koust6u koust6u merged commit a91965d into BE/dev Jul 24, 2024
12 checks passed
@koust6u koust6u deleted the koust6u-patch-1 branch July 24, 2024 06:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants