=== allenap changed the topic of #launchpad-reviews to: On call: allenap || reviewing: None || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === allenap changed the topic of #launchpad-reviews to: On call: allenap || reviewing: lunch-options || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [11:49] allenap: :-) [12:26] allenap: When you're done reviewing your lunch options, any chance I can add two more branches to your queue? [13:40] hi sinzui [13:40] hi bac [13:40] sinzui: could you have a look at the diff i just pasted to https://code.edge.launchpad.net/~bac/launchpad/bugjam-579502/+merge/44302 [13:40] i'd like to send this off to ec2 soon [13:46] bac: I see you changed _known_viewers() to ensure that owner have access [13:46] sinzui: yes, i wanted your feedback specifically on that approach. [13:46] bac: I see it always returns a set, but the doc string says dict [13:46] it seemed the least intrusive [13:47] I think we should fix the doc string [13:47] good catch! [13:47] bac: will that be the same spot to add drivers, and bug supervisors? [13:48] sinzui: with some careful review I think it might. [13:49] I think I will look at that method in january [13:50] bac; you are aware that the owner can only assign teams he is a member of to the bug supervisor role? [13:51] sinzui: i am but i don't see how that relates. what am i missing? [13:52] Every believes the owner can private bugs. But that is not true since he can leave the team, he could change the team, but that new team will not be subscribed to the bugs... [13:52] Your change makes Lp work as users expect...even Canonical employees [13:56] ah, a subtly i missed [13:56] er subtlety [13:58] sinzui: so, landable? [13:58] yes please [14:08] jelmer: Certainly :) === jcsackett changed the topic of #launchpad-reviews to: On call: allenap, jcsackett || reviewing: lunch-options || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === jcsackett changed the topic of #launchpad-reviews to: On call: allenap, jcsackett || reviewing: lunch-options, - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === allenap changed the topic of #launchpad-reviews to: On call: allenap, jcsackett || reviewing: -, - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === jelmer changed the topic of #launchpad-reviews to: On call: allenap, jcsackett || reviewing: -, - || queue: [jelmer]*3 || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [15:13] allenap, jcsackett, i have an launchpadlib branch coming up that's oversize, but mostly due to method renames [15:14] leonardr: Okay, I can take a look at that. [15:15] will let you know once i write mp === jcsackett changed the topic of #launchpad-reviews to: On call: allenap, jcsackett || reviewing: -, jelmer || queue: [jelmer]*2 || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [15:53] bac: can you follow up on https://code.launchpad.net/~jelmer/launchpad/693757-py2.7-compat/+merge/44566 for me? [15:53] jcsackett: sure [15:53] thanks. === jcsackett changed the topic of #launchpad-reviews to: On call: allenap, jcsackett || reviewing: -, - || queue: [jelmer]*2 || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === jcsackett changed the topic of #launchpad-reviews to: On call: allenap, jcsackett || reviewing: -, - || queue: [ ] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [16:17] allenap: https://code.launchpad.net/~leonardr/launchpadlib/separate-login-for-cronjob/+merge/44592 === allenap changed the topic of #launchpad-reviews to: On call: allenap, jcsackett || reviewing: leonardr, - || queue: [ ] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [16:19] leonardr: It says there's a conflict in src/launchpadlib/tests/test_launchpad.py === henninge_ is now known as henninge [16:24] jcsackett: Hi! I have to leave now but it would be great if you could look at my MP. It's fairly straight forward UI stuff. [16:24] henninge: i can look at the code portion, but i'm not a ui reviewer. === henninge changed the topic of #launchpad-reviews to: On call: allenap, jcsackett || reviewing: leonardr, - || queue: [henninge] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [16:24] jcsackett: that's fine [16:25] https://code.launchpad.net/~henninge/launchpad/devel-bugjamming-2/+merge/44590 [16:25] thank you! === jcsackett changed the topic of #launchpad-reviews to: On call: allenap, jcsackett || reviewing: leonardr, henninge || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [16:25] henninge: no problem. [16:42] allenap: hmmm, i just updated, let me check [16:42] ah, i updated but didn't merge [16:56] leonardr: Cheers. [16:56] allenap: should be fixed now === jelmer_ is now known as jelmer === allenap changed the topic of #launchpad-reviews to: On call: jcsackett || reviewing: henninge || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [17:35] bac: can you follow up on https://code.launchpad.net/~henninge/launchpad/devel-bugjamming-2/+merge/44590 for me, when you get a chance? === jcsackett changed the topic of #launchpad-reviews to: On call: - || reviewing: henninge || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [17:35] jcsackett: ok [17:35] thanks. [17:48] jcsackett: done [17:52] bac, thanks. :-) [17:52] although, on the MP, i see that sinzui is still doing reviews, so perhaps i can stop bothering you and pester him. === sidnei changed the topic of #launchpad-reviews to: On call: - || reviewing: henninge || queue: [https://code.launchpad.net/~sidnei/lazr-js/static-resource-prefix/+merge/44271, https://code.launchpad.net/~sidnei/lazr.testing/yeti/+merge/44591] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === jcsackett changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [https://code.launchpad.net/~sidnei/lazr-js/static-resource-prefix/+merge/44271, https://code.launchpad.net/~sidnei/lazr.testing/yeti/+merge/44591] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [19:18] sidnei: it works better to get reviewers attention if you leave a message in the room with the MPs and put your name in the queue than just throwing the MPS in the queue in the channel info. [19:22] i've got to run out to lunch, i'll be back here for reviews in a bit. === jcsackett changed the topic of #launchpad-reviews to: On call: jcsackett || reviewing: late lunch || queue: [https://code.launchpad.net/~sidnei/lazr-js/static-resource-prefix/+merge/44271, https://code.launchpad.net/~sidnei/lazr.testing/yeti/+merge/44591] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [19:48] jcsackett, i couldn't remember which way it was, thanks! === sidnei changed the topic of #launchpad-reviews to: On call: jcsackett || reviewing: late lunch || queue: [sidnei] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === jcsackett changed the topic of #launchpad-reviews to: On call: jcsackett || reviewing: sidnei || queue: [sidnei] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === jcsackett changed the topic of #launchpad-reviews to: On call: jcsackett || reviewing: sidnei || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [21:19] bac: i've done sidnei's first MP: https://code.launchpad.net/~sidnei/lazr-js/static-resource-prefix/+merge/44271. can you follow up? === jcsackett changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews