=== salgado-afk is now known as salgado === salgado is now known as salgado-afk === adeuring changed the topic of #launchpad-reviews to: on call: adeuring || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com [09:55] jml: Are you still part of the OCR rotation? In any case, https://dev.launchpad.net/ReviewerSchedule still has you down for AsicPac. [10:02] allenap, I'm taking a temporary leave of absence [10:03] allenap, I'll reschedule myself sometime soon. === mrevell is now known as mrevell-lunch [12:21] adeuring: hi, I've got a branch up for review, can you take it? :) === danilos changed the topic of #launchpad-reviews to: on call: adeuring || reviewing: - || queue: [danilo] || This channel is logged: http://irclogs.ubuntu.com [12:21] danilos: sure [12:22] adeuring: thanks, it's https://code.edge.launchpad.net/~danilo/launchpad/bug-435398/+merge/13462 (I've hit a bug submitting via email, so diff should be there in a minute or two if not already up) === adeuring changed the topic of #launchpad-reviews to: on call: adeuring || reviewing: danilo || queue: [] || This channel is logged: http://irclogs.ubuntu.com [12:52] danilos: nice work! just one question about a change you did not describe: In line 205 of the diff you remove "/+translations" from a link. Is this intentional, and if so, why? [12:53] adeuring: yes, it doesn't make much difference in practice, because that's the default anyway, but would allow us to change the default easier in the future without worrying of breaking unrelated tests (such as this one) [12:54] danilos: ah, thanks. So, r=me === adeuring changed the topic of #launchpad-reviews to: on call: adeuring || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com === noodles775 changed the topic of #launchpad-reviews to: on call: adeuring || reviewing: - || queue: [noodles] || This channel is logged: http://irclogs.ubuntu.com [12:56] Hi adeuring! I've got a straight-forward view/template change that should arrive in your inbox in a minute or two. [12:56] noodles775: ok [12:57] adeuring: how are you going with those hwdb changes? Looks like a huge change (great to see lots of small branches!) [12:57] it's been going for a couple of weeks now right? [12:58] noodles775: I noticed that I made some embarassingly wrong assumptions in the test data, so I'm updating tests again ;) [12:58] noodles775: yes, I'm working on this tsuff since two weeks or so. I hope that the changes will be finished next week [12:58] adeuring: is there anyone else who's familiar with all the udev stuff that could review those type of things for you? [12:59] noodles775: i don't think so... [12:59] s/that/who [12:59] ok. [12:59] adeuring: thanks [13:01] adeuring: here it is: https://code.launchpad.net/~michael.nelson/launchpad/430459-link-build-failures-updates-portlet/+merge/13463 [13:01] thanks! [13:02] intellectronica: would you have time for a UI review? It's "quite straight forward". See ^^^ before you answer :) [13:02] noodles775: sure [13:02] Thanks! [13:04] noodles775: it looks really nice. the only problem i have with it is that the error icon looks just like the remove icon. i guess that's out of scope for this branch, though? [13:04] is this icon new, or something that is all over the place in soyuz? [13:05] intellectronica: do you mean the failed to build icon? [13:05] noodles775: yes [13:05] intellectronica: yes, according to https://edge.launchpad.net/+graphics it's specifically for build-failed... [13:06] so we use it wherever we indicate failed builds. (part of the new icons that landed during 3.0) [13:07] noodles775: well, it's worth asking beuno what he thinks of that critique of the icon. i know he's been focusing on icon semantics quite a lot lately. other than that the change looks really good, so ui=me* === mrevell-lunch is now known as mrevell [13:07] intellectronica: thanks. It's one of the set that beuno introduced a few months ago, but yep, it'd be good to let him know of that confusion. [13:13] noodles775: r=me [13:13] thanks adeuring ! [13:20] hey jml: just noticed that you're a ui* reviewer? Would you have 5mins to do a second ui* review for me (it already has ui*=intellectronical, but most of the other ui reviewers are sprinting/not here yet). [13:20] https://code.launchpad.net/~michael.nelson/launchpad/430459-link-build-failures-updates-portlet/+merge/13463 [13:21] s/tronical/tronica ;) [13:21] noodles775, sure thing. fwiw, I don't get it done in the next 30 mins, I won't be able to get around to it today. === matsubara-afk is now known as matsubara [13:22] _if_ I don't get it done, rather. [13:22] jml: thanks. np, if you're too busy - just leave it - I can grab one of the US guys later. [13:26] noodles775, well, I'd like to at least _try_ to do it :) [13:53] noodles775, sorry, I've failed. [13:53] jml: no probs, thanks for trying :) [14:00] rockstar: are you able to do a second ui review for me? [14:00] https://code.launchpad.net/~michael.nelson/launchpad/430459-link-build-failures-updates-portlet/+merge/13463 [14:13] adeuring: i'm not sure how much reviewing salgado and i are going to do today. we're sprinting. if it's okay with you i'd like to avoid non-sprint related reviews at least until you go offline [14:13] adeuring: but ping me if the queue gets too long [14:13] barry: OK, thnkas for the heads-up. But up to now it was a really quit day [14:35] adeuring: could you look at another trivial branch for me? [14:35] https://code.edge.launchpad.net/~michael.nelson/launchpad/219222-edit-ppa-dependencies-rename/+merge/13468 [14:36] noodles775: sure [14:36] intellectronica: would you mind doing a ui review for the above too? (sorry, no one else is around) [14:36] adeuring: thanks. [14:37] s/around/around or available due to sprints etc. [14:48] adeuring: may i add a small branch to your queue? [14:48] bac: sure [14:48] adeuring: great. i'll paste the MP when it arrives === bac changed the topic of #launchpad-reviews to: on call: adeuring || reviewing: - || queue: [noodles,bac] || This channel is logged: http://irclogs.ubuntu.com [14:50] noodles775: r=me [14:50] Thanks again adeuring. [14:50] welcome :) === adeuring changed the topic of #launchpad-reviews to: on call: adeuring || reviewing: bac || queue: [bac] || This channel is logged: http://irclogs.ubuntu.com [15:06] bac: bug 436986 says that the map is already limited to 24 items. You set self.limit = 24 for class TeamMapLtdData. Couldn't you recycle the constant mentioned in the bug? [15:06] Bug #436986: Person:+index (map_portlet_html) timeout [15:07] adeuring: the limit mentioned in the bug was being realized at line 62 of the diff, but it did not work [15:08] adeuring: i should have a constant so i don't re-use 24 at lines 84 and 99 [15:08] bac: Ah, thanks, I missed that line in the diff. And right, please use a constant for both classes. [15:09] gladly [15:12] bac: one more question: your tests do not really show that the map is indeed limited to 24 items. Could you change the test in team-map.txt so that we really see that no more than TeamMap.limit items are returned? (probably with a modified version of TeamMap, where limit is 2) [15:13] ...erm, I meant class TeamMapData.... [15:14] adeuring: i should have worded that test more clearly. i'd like to use it to show the URL works but not necessarily the limit. the limit is exercised in the team-views.txt test. with additional explanation in the first do you think that'd be sufficient? [15:15] bac: well, you show the constant there, but you don't show that the constant has any effect ;) [15:15] which there? [15:15] in team-views.txt the constant is there. it is then set to 1 to show it does have affect? [15:15] bac: Ahhh, I missed that. [15:16] bac: thanksfor the explanantions. r=me [15:16] adeuring: np. thanks for the review! [15:25] with non-batched email from LP it is weird to see the email arrive before the web page refreshes when you update a MP === stub1 is now known as stub === adeuring changed the topic of #launchpad-reviews to: on call: adeuring || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com === sinzui changed the topic of #launchpad-reviews to: on call: adeuring || reviewing: - || queue: [sinzui] || This channel is logged: http://irclogs.ubuntu.com === Ursinha is now known as Ursinha-lunch [16:48] adeuring: barry: I have a mechanical branch for review: https://code.edge.launchpad.net/~sinzui/launchpad/package-link-validation/+merge/13484 [16:48] sinzui: I'll look === adeuring changed the topic of #launchpad-reviews to: on call: adeuring || reviewing: sinzui || queue: [] || This channel is logged: http://irclogs.ubuntu.com [16:49] adeuring: thanks. i'm doing a hairy one for gary on #launchpad-sprint (barry does hairy gary branch? ewwee) [16:49] heh === beuno is now known as beuno-lunch === matsubara is now known as matsubara-lunch [17:15] sinzui: r=me [17:21] adeuring: thank you very much [17:22] sinzui: welcome :) [17:35] adeuring: you have time for another or should i wait for barry? it's complicated by the fact i'm about to leave for lunch. [17:35] bac: one more branch is fine, but after that I'll leavem I think ;) [17:36] hey barry are you going to be reviewing after lunch? === adeuring changed the topic of #launchpad-reviews to: on call: adeuring || reviewing: bac || queue: [] || This channel is logged: http://irclogs.ubuntu.com [17:38] thanks abel. i'll answer any questions you have when i return. === Ursinha-lunch is now known as Ursinha === salgado is now known as salgado-lunch [18:14] bac: r=me === matsubara-lunch is now known as matsubara [18:25] adeuring: Could you please review https://code.launchpad.net/~abentley/launchpad/ignore-md-diff/+merge/13491 ? [18:26] abentley: sure === adeuring changed the topic of #launchpad-reviews to: on call: adeuring || reviewing: abentley || queue: [] || This channel is logged: http://irclogs.ubuntu.com [18:28] adeuring: Thanks. Btw, have you tried the inline reviewing? [18:29] abentley: no. How do I do this? [18:30] There is a comment/review box below the comments. [18:32] abentley: ah, that's what you meant ;) I'll try it [18:34] abentley: r0me [18:34] adeuring: Thanks! === adeuring changed the topic of #launchpad-reviews to: on call: - || reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com === salgado-lunch is now known as salgado [19:00] thanks adeuring === beuno-lunch is now known as beuno [20:43] hi beuno, can you do a UI review? i've got screenshots! [20:43] bac, hit me [20:43] https://code.edge.launchpad.net/~bac/launchpad/bug-452491-captcha2-boogaloo/+merge/13487 [20:44] beuno: it's adding a captcha to the 'i forgot my password' page. also fixes some spacing on the error message from the login page [20:45] bac, why are the forms different than everywhere else? [20:45] where the label is on top [20:45] I know, I know, nothing to do with your branch [20:45] beuno: don't know. these pages do not use LaunchpadFormView -- they do everything by hand. a huge PITA [20:46] bac, gotcha [20:46] beuno: converting them to LPFV would be a good tech-debt branch [20:46] approved === flacoste_afk is now known as flacoste === salgado is now known as salgado-phone === salgado-phone is now known as salgado === salgado is now known as salgado-afk === matsubara is now known as matsubara-afk