ci(workflows): send new issue notifications to app maintainers by app label#59752
ci(workflows): send new issue notifications to app maintainers by app label#59752edward-ly wants to merge 1 commit into
Conversation
cf263e9 to
c43685b
Compare
… label Signed-off-by: Edward Ly <contact@edward.ly>
c43685b to
e693248
Compare
|
|
|
Another potential alternative I just thought of would be to move each app out to separate repositories and re-add them as submodules instead, although I'm not sure yet if that'll affect other parts of the server in any way. |
|
fyi @miaulalala |
miaulalala
left a comment
There was a problem hiding this comment.
Makes sense to me, we should give it a trial run if it helps.
|
But experts are not app maintainers. For example look at the files app, they both will drown in notifications. That is the reason why we have github-teams for pull requests which assign people from the respective team in round-robin. While this might work for any other repository I highly doubt this will work for server. Please note that we already have a triagingWe already triage the server issues: 🧑🤝🧑 Community triage (view) If your point is that you think some app owners should be pinged about their apps then, we can add this to the process and let them handle them their issues. But at least for the server team thats not how work is split. |
Fully agree with these two points. |
|
The use case is that there are some server apps that are maintained by people in other teams. For example the comments app (maintained by Edward) and webhook listeners (maintained by Jana). So they need to get informed when there is a new issue for them. This must be a problem for many app maintainers that are not part of the files team. It's fine by me if this can be resolved by adjusting the triage process |
Summary
Whenever a new issue for a particular app in this repository is created, this new workflow will send a GitHub notification to the appropriate app maintainer(s) by posting a comment in the issue mentioning them.
The current list of eligible apps only includes those for which a corresponding label already exists, but I am open to suggestions regarding what the final list should contain.
cc @DaphneMuller @janepie
Checklist
3. to review, feature component)stable32)AI (if applicable)