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

[REFACTOR] /noti 워크플로우 수정 #13

Merged
merged 10 commits into from
Dec 12, 2024
Merged

[REFACTOR] /noti 워크플로우 수정 #13

merged 10 commits into from
Dec 12, 2024

Conversation

unifolio0
Copy link
Contributor

🚩 연관 이슈

closed #12

🗣️ 리뷰 요구사항 (선택)

job을 리뷰코멘트용이랑 코멘트용으로 분리했습니다

@unifolio0 unifolio0 added the refactor 리팩토링 label Dec 10, 2024
@unifolio0 unifolio0 self-assigned this Dec 10, 2024
Copy link
Contributor

@leegwichan leegwichan left a comment

Choose a reason for hiding this comment

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

/noti

기능에는 문제 없을 것 같아 Approve 했습니다~
추가적으로 의견 남겼습니다~!

Comment on lines 77 to 81
run: |
echo "AVATAR_URL=${{ secrets.DISCORD_AVATAR_URL }}" >> $GITHUB_ENV
echo "COMMENT_BODY= 🤚백엔드 친구들 모여라~ 🔊" >> $GITHUB_ENV
echo "USERNAME=망나뇽" >> $GITHUB_ENV
echo "WEB_HOOK=${{ secrets.DISCORD_WEB_HOOK }}" >> $GITHUB_ENV
Copy link
Contributor

Choose a reason for hiding this comment

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

[제안]

  • "망나뇽" 딴 걸로 바꾸실?
  • COMMENT_BODY 안 넣는게 오히려 좋을 수도?
    • 지금 알림을 보면 '어 이거(🤚백엔드 친구들 모여라~ 🔊) 왜있지?' 라는 생각이 들어서

Copy link
Contributor

Choose a reason for hiding this comment

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

[커찬의 제안에 대한 의견]

  • 망나뇽 바꾸는 것 찬성합니다 ㅋㅋㅋ 디코봇 정도 네이밍은 어떠신가요?
  • 저도 유의한 comment body는 넣어야 한다는 것에 찬성해요. 지금은 어떤 PR에 코멘트를 남겼는지 구분이 되지 않는다는 단점이 있었던 것 같아서, PR 타이틀을 넣어보는 건 어떨까요?
Suggested change
run: |
echo "AVATAR_URL=${{ secrets.DISCORD_AVATAR_URL }}" >> $GITHUB_ENV
echo "COMMENT_BODY= 🤚백엔드 친구들 모여라~ 🔊" >> $GITHUB_ENV
echo "USERNAME=망나뇽" >> $GITHUB_ENV
echo "WEB_HOOK=${{ secrets.DISCORD_WEB_HOOK }}" >> $GITHUB_ENV
run: |
echo "AVATAR_URL=${{ secrets.DISCORD_AVATAR_URL }}" >> $GITHUB_ENV
echo "COMMENT_BODY= github.event.issue.pull_request.title 리뷰 완료!" >> $GITHUB_ENV
echo "USERNAME=디코봇(더 좋은 이름이 있다면 그걸로!)" >> $GITHUB_ENV
echo "WEB_HOOK=${{ secrets.DISCORD_WEB_HOOK }}" >> $GITHUB_ENV

Copy link
Contributor Author

Choose a reason for hiding this comment

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

디코봇, 해당 PR or Issue의 타이틀로 수정했습니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

스크린샷 2024-12-12 오후 5 57 12 예시 사진입니다!

coli-geonwoo
coli-geonwoo approved these changes Dec 11, 2024

jobs:
notify-discord:
notify-discord-comment:
Copy link
Contributor

Choose a reason for hiding this comment

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

오... 자동화 actions를 자세히 뜯어보는 건 오늘이 처음인데, 그럼 꼭 상단에 /noti 를 적지 않아도 트리거가 발동되겠군요! 저는 지금까지 상단에 적어야 하는 건 줄 알았어요 😅

Copy link
Contributor Author

Choose a reason for hiding this comment

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

그냥 들어있기만 하면 상관없습니다!

Comment on lines 77 to 81
run: |
echo "AVATAR_URL=${{ secrets.DISCORD_AVATAR_URL }}" >> $GITHUB_ENV
echo "COMMENT_BODY= 🤚백엔드 친구들 모여라~ 🔊" >> $GITHUB_ENV
echo "USERNAME=망나뇽" >> $GITHUB_ENV
echo "WEB_HOOK=${{ secrets.DISCORD_WEB_HOOK }}" >> $GITHUB_ENV
Copy link
Contributor

Choose a reason for hiding this comment

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

[커찬의 제안에 대한 의견]

  • 망나뇽 바꾸는 것 찬성합니다 ㅋㅋㅋ 디코봇 정도 네이밍은 어떠신가요?
  • 저도 유의한 comment body는 넣어야 한다는 것에 찬성해요. 지금은 어떤 PR에 코멘트를 남겼는지 구분이 되지 않는다는 단점이 있었던 것 같아서, PR 타이틀을 넣어보는 건 어떨까요?
Suggested change
run: |
echo "AVATAR_URL=${{ secrets.DISCORD_AVATAR_URL }}" >> $GITHUB_ENV
echo "COMMENT_BODY= 🤚백엔드 친구들 모여라~ 🔊" >> $GITHUB_ENV
echo "USERNAME=망나뇽" >> $GITHUB_ENV
echo "WEB_HOOK=${{ secrets.DISCORD_WEB_HOOK }}" >> $GITHUB_ENV
run: |
echo "AVATAR_URL=${{ secrets.DISCORD_AVATAR_URL }}" >> $GITHUB_ENV
echo "COMMENT_BODY= github.event.issue.pull_request.title 리뷰 완료!" >> $GITHUB_ENV
echo "USERNAME=디코봇(더 좋은 이름이 있다면 그걸로!)" >> $GITHUB_ENV
echo "WEB_HOOK=${{ secrets.DISCORD_WEB_HOOK }}" >> $GITHUB_ENV

@coli-geonwoo
Copy link
Contributor

/noti

비토, review용 comment와 일반 코멘트의 워크플로를 모두 트리거로 지정해주어 이젠 두 상황 모두에서 코멘트 웹 훅이 트리거되겠네요! 사소한 부분이었는데 신경써주어 감사합니다 🙇

커찬 의견과 관련하여 사소한 의견 남겼으니 반영은 비토가 자유롭게 해주세요!

@unifolio0
Copy link
Contributor Author

/noti
리뷰 반영 및 수정사항 추가했어요~ 한번만 더 확인해주세요~

@unifolio0
Copy link
Contributor Author

/noti
이미 approve 되어있긴한데 확인만해주세요
이제 /noti하면
image
이런 식으로 알림이 옵니다!

Copy link
Contributor

@leegwichan leegwichan left a comment

Choose a reason for hiding this comment

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

/noti 확인 완료!

@unifolio0 unifolio0 merged commit bcd13a8 into develop Dec 12, 2024
@unifolio0 unifolio0 deleted the refactor/#12 branch December 12, 2024 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 리팩토링
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[REFACTOR] /noti 워크플로우 수정
3 participants