Merge ~racb/ubuntu-sponsoring:ubuntu-sponsors-reporter into ubuntu-sponsoring:main

Proposed by Robie Basak
Status: Needs review
Proposed branch: ~racb/ubuntu-sponsoring:ubuntu-sponsors-reporter
Merge into: ubuntu-sponsoring:main
Diff against target: 12 lines (+1/-1)
1 file modified
sponsors-page (+1/-1)
Reviewer Review Type Date Requested Status
Benjamin Drung Needs Fixing
Ubuntu Development Team Pending
Review via email: mp+460697@code.launchpad.net

Description of the change

See commit message.

Do you want this to go into the develop branch first instead?

To post a comment you must log in.
Revision history for this message
Benjamin Drung (bdrung) wrote :

Have you tested the change locally?

review: Needs Fixing

Unmerged commits

d149c0a... by Robie Basak

Add ~ubuntu-sponsors-reporter to SPONSOR_TEAMS

This will cause MPs for which ~ubuntu-sponsors-reporter has a review
slot (or a bug subscription, but we don't care about that) to also
appear in the sponsorship report.

I will also change the bot that auto-adds ~ubuntu-sponsors to add
~ubuntu-sponsors-reporter instead.

Sponsors will then be able to vote on MPs without "grabbing" the
~ubuntu-sponsors-reporter review slot (since they intentionally won't
belong to that team), and this way the MP won't disappear from the
sponsorship queue when that happens.

See
https://discourse.ubuntu.com/t/improving-the-git-ubuntu-based-sponsorship-workflow/37964
for further discussion.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/sponsors-page b/sponsors-page
2index 13bf542..44c24b3 100755
3--- a/sponsors-page
4+++ b/sponsors-page
5@@ -33,7 +33,7 @@ EXTRA_TEAMS = {
6 # Items against packages in b should go to a's page
7 SETS = {"desktop-core": "ubuntu-desktop"}
8
9-SPONSOR_TEAMS = {"general": "ubuntu-sponsors", "security": "ubuntu-security-sponsors"}
10+SPONSOR_TEAMS = {"general": "ubuntu-sponsors", "ubuntu-sponsors-reporter", "security": "ubuntu-security-sponsors"}
11
12 BLACKLISTED_REQUESTER = ("package-import",)
13

Subscribers

People subscribed via source and target branches