[00:44] <bac> good morning LP
[00:44] <bac> bring out reviews if you need them
[04:15] <bac> thumper: for a mentored review use "code" as the type.  i don't think our tools understand "mentor".  let me know if i'm wrong.
[04:16] <thumper> bac: ah, ok
[04:33] <lifeless> bac: Do they understand mentoring at all ?
[04:35] <bac> lifeless: the last time i looked they knew about code and code*, converting them to [r=mentat,mentor]
[04:35] <bac> i don't think they will reject a code* with no code
[04:35] <lifeless> interesting
[04:35] <lifeless> thanks
[08:46] <jtv1> hi bac!  Got one coming up for ya
[08:46] <jtv1> Nice, simple one.
[08:46] <jtv1> What could go wrong, right?  :-)
[08:46] <bac> ok, jtv
[08:46] <jtv> (still generating diff)
[08:47] <jtv> bac: there it is!  https://code.launchpad.net/~jtv/launchpad/recife-translationpolicy/+merge/40165
[08:47] <jtv> Gave it a nice political twist.
[09:14] <jtv> hi adeuring!
[09:15] <jtv> (more accurate I believe)
[10:04] <bac> adeuring: i'm going to leave the reviewing to you for the day.
[10:04] <adeuring> bac: have a nice weekend!
[10:04] <bac> you too, abel.
[10:04] <adeuring> thanks!
[11:15] <jelmer> adeuring: Hi!
[11:15] <jelmer> adeuring: Can I add a MP to your queue?
[11:15] <adeuring> jelmer: sure
[11:15] <jelmer> The MP is at https://code.launchpad.net/~jelmer/launchpad/syncpackagejob-1/+merge/40169
[11:48] <adeuring> jelmer: your TODO comment in getctiveJobs() says that you don't expect a larger result set for the find() call. What about logging a warning if the result set is greater than a reasonable threshold?
[11:48] <jelmer> adeuring: That makes sense.
[11:49] <adeuring> jelmer: great, so, r=me.
[11:49] <jelmer> adeuring: Thanks!
[15:19] <jtv> adeuring: got a simple one for you… https://code.launchpad.net/~jtv/launchpad/enum-translationpermission/+merge/40192
[15:21] <jtv> Whoops, and just like that I spot an __all__ line I forgot to clean up.  Pushing change now.
[15:26] <adeuring> jtv: I'll look
[15:27] <jtv> thanks!
[15:40] <adeuring> jtv: approved
[15:50] <abentley> rockstar: I can has review? https://code.launchpad.net/~abentley/launchpad/resubmit-change-branch/+merge/40126
[16:11] <rockstar> abentley, looking
[16:13] <jtv> thanks adeuring
[18:30] <jelmer> jml: is there a bug for the mp you just submitted? I'm just curious about the reasoning behind it.
[18:30] <jml> jelmer: I couldn't find one
[18:30] <jml> jelmer: we don't need to have a coc signed before allowing uploads.
[18:30] <jml> jelmer: ergo, the code can be deleted
[18:32] <jelmer> jml: I guess that's what I'm curious about, why is having the coc signed not necessary?
[18:32] <jml> jelmer: why is it necessary?
[18:34] <jelmer> jml: PPA's are specific to Ubuntu, and as such I think the CoC is relevant.
[18:34] <jelmer> jml: people who create and maintain PPA's are ubuntu community members
[18:34] <jml> jelmer: some of them are
[18:37] <jelmer> jml: Hmm
[18:38] <jml> jelmer: anyway, I plan on talking about it w/ you & bigjools next week
[18:38] <jelmer> jml: ah, ok. I can wait :-)
[18:38] <jml> jelmer: I just figured I'd put the patch up since it's fairly simple, and if I talked first it would have ended up on a queue of bugs rather than a queue of patches
[18:39] <jelmer> jml: Makes sense.
[18:40] <jelmer> jml: Enjoy your weekend! talk to you next week.