=== matsubara-afk is now known as matsubara === matsubara is now known as matsubara-afk === jtv-afk is now known as jtv === henninge_ is now known as henninge [10:35] henninge: do you want this one? https://code.launchpad.net/~jtv/launchpad/recife-another-testfix/+merge/29262 [10:35] jtv: anything to get this merge done ... ;) [10:35] henninge: the fix should work equally with or without the merge, so should be easy. [10:39] hi danilos [10:39] jtv, hi [10:47] danilos: hi ;) [10:47] jtv: you are talking "product series" but the change is about "distro series". Why is that? [10:48] henninge: probably a simple slipup... where exactly? [10:48] jtv: in the mp description [10:49] henninge: oh, then it's definitely a typo. Finger memory. [10:49] ok [10:49] let me just verify that htis helps ... ;) [10:49] As you would have guessed from factory.makeProductSeries(distribution=ubuntu) :-) [10:58] jtv: great. it works! [10:58] jtv: r=me [10:58] henninge: dankeschön! [11:14] jtv: will you merge it into recife right away? [11:14] henninge: it's currently pushing [11:15] great! :-D [11:16] It Is Pushed™ [11:17] I just remembered that there were also windmill tests failing. [11:19] henninge: got a log? [11:20] jtv: yes, it's in the same [11:20] http://people.canonical.com/~henninge/merged-lp.translations.tests.log === gmb changed the topic of #launchpad-reviews to: On call: gmb || reviewing: lunch || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === mrevell is now known as mrevell-luncheon === matsubara-afk is now known as matsubara === gmb changed the topic of #launchpad-reviews to: On call: gmb || reviewing: - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === bigjools changed the topic of #launchpad-reviews to: On call: gmb || reviewing: - || queue: [bigjools] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [13:19] * bigjools waves at gmb [13:19] Hullo [13:19] bigjools, Diff me. [13:20] I just 'bzr send'ed the MP, it should appear shortly [13:20] RIghto [13:20] https://code.edge.launchpad.net/~julian-edwards/launchpad/oops-dspr-page-bug-592417/+merge/29273 [13:38] bigjools, r=me === gmb changed the topic of #launchpad-reviews to: On call: gmb || reviewing: - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [13:39] thanks gmb. And see yer next week. [13:39] Would've been quicker but I had a tea-based emergency [13:39] bigjools, Welcome. And likewise, unless I go blind in the interim. [13:39] as in you needed more [13:39] or as in you spilt it [13:39] bigjools, It had gone cold. [13:40] oh disaster [13:40] Indeed. Calamity, oh woe is me, etc. [13:40] Thankfully, it has now been refreshed. Hurrah, etc. Let joy be unconfined. [13:40] and on that note I need coffee. Some Finca el Fany awaits. [13:44] bigjools: I've marked the mp as ui=me, but with a small suggestion - see what you think. [13:45] noodles775: ok cheers [13:47] noodles775: see https://dogfood.launchpad.net/ubuntu/+source/uex/1.0.0.9-1/+index [13:47] with that listing change [13:48] I agree that the old way looks a bit odd, but this way looks more odderer :) [13:50] noodles775: a third option to add to your suggestions would be to centre the data across the three s [13:54] bigjools: there are other class (and class combinations) for tables in the stylesheet(s)... I'd recommend going with one of those (as they're what have been developed for consistency). [13:54] I think centering the data inside the td's would be odderer still ;) [13:54] true [13:54] :) [13:55] noodles775: remind me where the css files live .... === mrevell-luncheon is now known as mrevell [13:56] ah found 'em [14:01] Great. [14:01] noodles775: unfortunately nothing looks that good [14:02] * noodles775 has a look at product and other pages. [14:03] bigjools: every table I can find is using the listing class? [14:04] That doesn't mean it looks good ;), but does mean that it would be best to use it and if we later improve the look of table.listing, they'll all be improved together... or is it something particular about this instance that you didn't like? [14:04] noodles775: ok what about now? [14:05] What don't you like about it? (and definitely not if you're going to land this RC ;) 0. [14:05] interestingly FF doesn't render the end of the "Size" heading cell marker [14:05] it's too wide [14:05] it only needs to fill the space it needs [14:06] IMO [14:08] That's what the 'narrow' class is for in the stylesheet... not sure why it's not doing anything though. [14:08] there's a "compressed" as well but it doesn't do anything either [14:09] noodles775: aha, check it now [14:09] not sure about the "-" characters [14:10] maybe just leave it blank [14:10] hmm... it's still wide for me (I've shift-refreshed). What classes did you add? [14:11] noodles775: it renders differently on Chrome and FF :/ [14:11] According to the chromium inspector, the 'narrow' isn't doing anything... [14:11] "narrow listing" [14:11] ah. [14:11] * noodles775 looks in FF [14:11] * bigjools shakes fist at browsers [14:12] bigjools: so right, I think you've found the right classes, and up to you whether you leave the '-' in or not. [14:13] Might be worth checking with sinzui or creating a bug about the 'narrow' class seemingly being ignored on chromium. [14:13] (just in case he already is aware of it). [14:13] yeah, I'll stick with "narrow" then and file the bug [14:13] cheers === mars changed the topic of #launchpad-reviews to: On call: gmb, mars || reviewing: - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === sinzui changed the topic of #launchpad-reviews to: On call: gmb, mars || reviewing: - || queue: [sinzui] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [14:55] gmb, mars: I have a branch that removes lots of crufty lint checking [14:56] sinzui, I'll take it === mars changed the topic of #launchpad-reviews to: On call: gmb, mars || reviewing: -,sinzui || queue: [sinzui] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [15:00] sinzui, out of curiosity, does pylint have a test suite? [15:00] specifically, a test suite with sample data [15:01] I do not think it does [15:02] It does not have a public bug tracker [15:03] sinzui, does your linter choke on conflict markers? '<<<' and '>>>' [15:03] no [15:03] does it flag them? :) [15:03] It looks for 7 start or close, which is better than what we had [15:04] 7 start or close? [15:04] mars: our linter reports the start of a conflict marker. pocketlint report the start and the end [15:04] ah, ok [15:06] sinzui, why was jslint was removed from the list of checks? Do we have a backup? [15:06] pocket-lint uses spider-monkey too [15:07] it however reports problems consistently with all the other linters [15:08] sinzui, a simple test should verify that - remove a semicolon from a JS file, see if it catches it [15:08] yep [15:09] I did not even know we had this lp-verbose.py stuff [15:09] I do [15:09] it's like rummaging through an attic [15:09] You could run lint.sh manually to do a serious analysis of modules to locate duplicate code [15:23] sinzui, reviewed === mars changed the topic of #launchpad-reviews to: On call: gmb, mars || reviewing: -,- || queue: [sinzui] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === mars changed the topic of #launchpad-reviews to: On call: gmb, mars || reviewing: -,- || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [15:24] thanks [15:38] mars, python-pocket-lint is a launchpad-developer-dependency. Did you update today and do you see that we guarantee the file? [15:39] update-manager died with a system crash, so no, I missed that :p [15:41] How ironic is it that change that come from our PPA are not "a source that supports changelogs" [15:41] sinzui, ok, got it, thanks. You may want to tell the lp-dev list after this lands. [15:42] mars: hysterical raisins [15:43] mars, I certainly will. I get to end two action items from the reviewers and team lead meetings [15:43] nice === matsubara is now known as matsubara-lunch === gmb changed the topic of #launchpad-reviews to: On call: mars || reviewing: - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [17:31] gary_poster: fancy a review of my branch "make LFA.restricted changable"? https://code.edge.launchpad.net/~adeuring/launchpad/bug-39674-lfa-editable/+merge/29314 [17:31] adeuring: what could you possibly be talking about? ;-) Sure, looking [17:31] gary_poster: thanks! === matsubara-lunch is now known as matsubara === jtv is now known as jtv-afk === benji is now known as benji-lunch [17:55] approved, adeuring [17:56] gary_poster: thanks! [17:56] np :-) [18:04] gary_poster: thanks for your suggestion to use assertRaises(setarr...) [18:04] adeuring: welcome, glad you like it === salgado is now known as salgado-lunch === benji-lunch is now known as benji === salgado-lunch is now known as salgado === salgado is now known as salgado-afk === matsubara is now known as matsubara-afk [22:29] mars are you around much longer for another review? [22:29] bac, gah, no, sorry, being burned by the release now :( === mars changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [22:29] ah right [22:30] no problem