Too many Email Notifications since Committers (Maintainers) seem to automagically get added as reviewers now? #2320
Comments
vorburger
added
type:bug
an example of that is e.g. #2319 |
|
I'm also getting pop-up notifications from some (work internal, I think) tool e.g. for 2319, because, even though I (manually, which I shouldn't have to) clicked Unsubscribe on it, and it says You’re not receiving notifications from this thread. on its Notifications, on top it still says "Xyz requested your review on this pull request." so that notification tool probably still think it should tell me e.g. when the PR is updated with new changes. |
|
This is now annoying (interrupting) me enough that I want to fix it! I have figured out that it's because of https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners, ergo specifically the mention of Line 2 in 63151b5 This means that the current 11 members of https://github.com/orgs/google/teams/android-fhir are all pinged to review PRs. But the reality is that those 11 do not. There appears to be a separate https://github.com/orgs/google/teams/android-fhir-reviewers - let's use that one! #2536 raised to proposed to change it. |
…viewers of PRs This initial list matches https://github.com/orgs/google/teams/android-fhir-reviewers. We were not able to get @google/android-fhir-reviewers working; see #2536 for background. See #2320 for overall motivation.
…viewers of PRs (#2545) This initial list matches https://github.com/orgs/google/teams/android-fhir-reviewers. We were not able to get @google/android-fhir-reviewers working; see #2536 for background. See #2320 for overall motivation.
It "feels" to me like I may be getting too many 💌 Email Notifications from this repository...
Some are normal and expected, as I contribute (more), GitHub notifies me about Comment - great!
But there are also notifications which I do not expect to receive, and typically don't get on other open source projects... as far as I can tell so far, they may break down into 1-2 different cases:
There seems to be something (new?) which automagically decides that I could be a suitable reviewer on certain PRs? An example is Allow modification of Questionnaire submit action button text via configuration #2319 which says "DebbieArita requested a review from vorburger (assigned from google/android-fhir)" - was that an intentional recent repo Settings configuration change? Or something new that's https://github.com/google wide? I'm not sure I'm a huge fan of that... 😃
I am not 100% sure if there is something else, slightly different from above, where thinks that GitHub may think it should send me email for PRs where the
android-fhirGroup (not me individually) is added as Reviewer; but I don't have an example of that yet.@jingtang10 my I assign this to you to look into? If you want I can also help to figure this out. Reading https://docs.github.com/en/organizations/organizing-members-into-teams/managing-code-review-settings-for-your-team, it looks like that's something that we should be able to tweak on the Settings tab on https://github.com/orgs/google/teams/android-fhir, but I don't seem to have access to that - do you?
If this is less of a "technical tool configuration" and more of a general "process" kind of discussion, I'm also happy to chat about how the project can best go about this, based on what I have seen in other open source communities. One feedback I would have is to perhaps establish a clearer sub-group of active general reviewers (of which e.g. I myself am not sure I could be a part of, both time and current skill and codebase familiarity wise), and then a clear process of roles and responsibilities and "cadence".
PS: If this can't be fixed by project configuration I would have to address it with personal email filtering; but if possible, it seems better to me to fix this in the project, and benefit other Committers (Maintainers) as well.
The text was updated successfully, but these errors were encountered: