-
Notifications
You must be signed in to change notification settings - Fork 750
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Update 'notify_team_new_comment' GitHub action to ignore comments by the Sentry bot #12568
Comments
Hi @MisRob MisRob! 👋 Is this issue open for contributions? |
Hi @josephinoo, I'm happy to assign this to you. I'm not exactly sure how to identify that the author is Sentry bot. Would you be able to research it and test it? |
Thanks, @MisRob , I'll look into it and if I find anything I'll let you know. |
Hi @josephinoo, I've just realized we will also need to ignore comments by Learning Equality bot such as this one would you like to include it in this issue, or should I open a new one? Meanwhile I realized there should be a simple way to figure out what's the bot's name, since we actually show it exactly as it is in the Slack message. Code for it is |
Hi @MisRob . |
Hi @Sanchit-100, since we didn't hear back from @josephinoo here and on other issues for several weeks, I will re-assign. Please see the comments above with the additional guidance. |
Thank you |
Hi @Sanchit-100 , are you working on this issue? |
Summary
Update 'notify_team_new_comment' GitHub action to ignore comments by the Sentry bot
Observed behavior
The GitHub action that sends a Slack notification when a contributor comments on issue, .github/workflows/notify_team_new_comment.yml, sends us notifications for comments by the Sentry bot
sentry-io[bot]
:Related comment: learningequality/studio#4643 (comment)
Expected behavior
notify_team_new_comment
is updated so that we don't receive notifications fromsentry-io[bot]
.Comments
Unless #12567 is resolved at first, note that copies of this action need to be updated in the same way in the following repositories:
The text was updated successfully, but these errors were encountered: