/srv/irclogs.ubuntu.com/2022/03/31/#launchpad.txt

rbasakWhy is list(lp.people['ubuntu-sponsors'].getRequestedReviews()) empty, even though https://code.launchpad.net/~ubuntu-sponsors/+activereviews has results?12:00
rbasakI'm looking to fix up the sponsoring page to display MPs again (but this time git-ubuntu rather than bzr).12:01
rbasakIt looks to me that the code should already work anyway, if I add ~ubuntu-sponsors as a reviewer to those branches.12:01
rbasakBut it's not, and I narrowed it down to getRequestedReviews() as above.12:02
cjwatsonBecause I never got round to figuring out the correct response to the review feedback on https://code.launchpad.net/~cjwatson/launchpad/git-getRequestedReviews/+merge/27113612:04
rbasakAh, OK. Thank you for working on this.12:06
rbasakRecently I've seen an increase in people filing MPs against git-ubuntu branches. I think people are assuming that if they submit such an MP then somebody will review it, but currently that doesn't happen.12:07
rbasakI was hoping to connect it up to the sponsoring page at least.12:07
rbasakFor now I was just going to have a bot add a ~ubuntu-sponsors review slot for every ~git-ubuntu-import review slot, and then abstain the ~git-ubuntu-import slot with an explanation.12:07
rbasakAnd then make sure that the sponsoring page displays those MPs like it used to for bzr MPs.12:08
rbasakI can't think of a way to do this without getRequestedReviews() :-/12:08
cjwatsonYeah, I think it probably does need that, but I'm not sure when I'll have time to work on this12:10
cjwatsonMaybe my comments are intelligible to some of my colleagues?12:10
jugmac00cough... :-)12:38
jugmac00we are all pretty busy - maybe we need to do some repriorization12:38
rbasakIt doesn't block me from anything. It's just a wart I was hoping to fix. So not massively urgent, but it'd be nice to be able to handle these MPs better.14:12
=== chrisccoulson_ is now known as chrisccoulson

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!