-
Notifications
You must be signed in to change notification settings - Fork 148
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
Support excluding specific comment authors for issue comment subscriptions #739
Comments
FYI: There is already a per-user setting in the form of |
@hanzei Funny enough, I had that in the issue description but removed it for some reason ![]() Are you suggesting we call the flag |
@hanzei I'd like to move this ticket to the monorepo to increase visibility. Do you have any thoughts on my question above? |
My concern is more about whether this setting should be per-user, per-subscription or global. There is already a per-user one. What value does a per-subscription setting add? |
@hanzei The per-user one is for DM notifications. How are you proposing this feature should work? I guess I'm simply not understanding what the exact suggestion is |
@hanzei Do you think this would work well? Or how do you see this working?
|
Comments from the
codecov
bot can be a bit spammy in channels subscribing to issue/PR comments. For my use case of the GitHub issue comments subscription, I would like to exclude thecodecov
bot from creating issue comments in the issue comment subscription.The task here is to implement a
--exclude-users
flag that can take a comma-separated list of GitHub usernames to exclude from the subscription posts. This is currently implemented for DM notifications through the mute command, though is not accessible in the subscriptions featuremattermost-plugin-github/server/plugin/command.go
Line 216 in 2608197
The text was updated successfully, but these errors were encountered: