=== _mup__ is now known as _mup_ [06:46] lifeless: Does https://code.launchpad.net/~stub/launchpad/update-storm/+merge/40264 need to go through review? [06:46] 2.4k of pretty much mechanical changes and delintings [06:55] stub: hi [06:57] uhm, I wouldn't, but I would get the *approach* discussed [06:57] stub: e.g. the cast in the core is a little concerning in general [07:06] lifeless: Didn't really seem to be any other sane way of doing it. I could have fixed the tests feeding raw strings in, but that could create too many brush fires on production with all our untested code paths (or even fixing some of our tests - gina.txt is utterly inscrutable). [07:07] gina.txt is a horrid test [07:07] sure [07:07] I guess the main thing to do is to behave sanely if a non-ascii str is passed in === bac changed the topic of #launchpad-reviews to: On call: henninge || Reviewing: allenap, - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [07:08] as well as have some way to let us incrementally report and fix things === bac changed the topic of #launchpad-reviews to: On call: henninge || Reviewing: allenap || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [07:08] the former should be done before landing for safety IMO [07:08] the latter whenever [07:08] lifeless: Where I have a unicode() cast, I could change that to something else that raises a warning, but I suspect that would be YAGNI. [07:09] I suggest this: [07:09] if type(thing) is not unicode: [07:09] try: [07:09] thing = thing.decode('utf8') [07:09] except UnicodeDecodeError: [07:09] bail [07:10] shove that in a helper [07:10] Oh - nothing should be getting UTF8 that I changed. We do want it to explode if we get non-ascii. [07:10] just doing a plain cast to unicode is a problem because it can do nuts things [07:10] stub: sure, s/utf8/ascii/ there [07:17] * stub wonders if there is an easy way to s/unicode/ensure_unicode/g in the lines changed in my branch [07:26] grep + interdiff might be able to do something [08:01] interdiff does the magic === henninge 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 === gmb changed the topic of #launchpad-reviews to: On call: gmb || Reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === matsubara-afk is now known as matsubara [11:33] bigjools, jtv: if either of you would be so kind (maybe a revie is optional here ... ;) [11:33] https://code.edge.launchpad.net/~henninge/launchpad/devel-difftacular/+merge/40419 [11:34] henninge: sounds like a perfect no-review branch to me [11:34] (without looking that is) [11:34] what is the tag [11:34] ? [11:34] [rs=henninge] ? [11:34] no idea, I'm waaaay behind on email [11:34] ;) [11:34] Something like r=noreview, but probably not that [11:35] rs=me ? [11:35] do the tools work with this brave new world? [11:35] henninge: too late, I've already approved it [11:35] w00t! [11:35] jtv: thanks [12:24] gmb: Hai! If you're still reviewing, I have a small branch ready for you === Ursinha-afk is now known as Ursinha [13:02] StevenK: Sure. Link me === mrevell is now known as mrevell-lunch [13:06] gmb: https://code.edge.launchpad.net/~stevenk/launchpad/move-ppaexception/+merge/40408 [13:06] * gmb looks [13:07] StevenK: 81 lines. My kinda branch. [13:07] Haha [13:07] StevenK: r=me === mars changed the topic of #launchpad-reviews to: On call: gmb, mars || Reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [13:10] StevenK, hi [13:11] morning gmb [13:11] Hi mars [13:14] mars: Hi [13:14] gmb, have you started into the activereviews pile? === mars changed the topic of #launchpad-reviews to: On call: gmb, mars || Reviewing: -,jml || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [13:19] mars: No, I haven't yet. [13:20] I will be doing shortly, though. [13:20] mars: Feel free to make a start and I'll pick up whatever you're not working on. [13:20] gmb, ok, I'll update the topic as I go [13:20] Cool [13:21] mars: Also, using "Claim review" takes it out of the list for the review team. [13:22] gmb, ah! thanks === mars changed the topic of #launchpad-reviews to: On call: gmb, mars || Reviewing: -,- || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === mrevell-lunch is now known as mrevell [14:55] gmb: care for a small one? The funny bits are in the larger follow-up branch though. https://code.launchpad.net/~jtv/launchpad/recife-pre-cleanups/+merge/40434 [14:56] jtv: Sure. [14:56] thx [15:01] jtv: r=me [15:01] gmb: thanks [15:09] * gmb goes off call for to do some hackering === gmb changed the topic of #launchpad-reviews to: On call: mars || Reviewing: - || queue: [] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [15:15] mars: can you take my big one? [15:15] After some wild swings, it came out at a neat 799 lines. [15:15] https://code.launchpad.net/~jtv/launchpad/recife-policy-invites-allows/+merge/40438 [15:17] jtv, I can, I'm helping deryck with something though, so it may be a bit before I can start it [15:17] first on the queue though [15:17] mars: great, thanks. I'll wander in and out for the coming few hours. === matsubara is now known as matsubara-lunch === jtv changed the topic of #launchpad-reviews to: On call: mars || Reviewing: - || queue: [jtv] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [15:38] (as agreed) === Ursinha is now known as Ursinha-doctor === matsubara-lunch is now known as matsubara === benji is now known as benji-lunch === Ursinha-doctor is now known as Ursinha === benji-lunch is now known as benji === benji changed the topic of #launchpad-reviews to: On call: mars || Reviewing: - || queue: [jtv, benji] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [19:16] mars: when you get a chance, here's a small MP for your perusal: https://code.launchpad.net/~benji/launchpad/bug-621090/+merge/40474 [19:17] thanks benji [19:22] sinzui, I haven't been getting any UI reviews lately. is it just me or are devs not working on UI lately? [19:23] I have had only one request in 3 weeks, and I could not do it [19:24] salgado, I think lose of edge is causing us to rethink ui. [19:25] sinzui, why's that? [19:26] I think people are learning feature flags. [19:26] I think weeks 4 and 1 were busiest for UI because we need to get UI changes on edge. [19:27] Since UDS was on a week 1, we may have had a natural decline in UI requests [19:28] yeah, that's possible [19:28] salgado, my team is at the end of a feature. so we have less UI work, but expect a lot in a few weeks [19:29] the code team will get a lot in a few weeks too because they will hack on blueprints [19:31] sinzui, cool, I'm looking forward to some UI reviews. [19:33] You will get lots. henninge, you, and myself are the UI reviewers [19:42] mars: hope you'll be able to get to my branch, because the next OCR shift is probably yours truly! [19:44] jtv, :) [19:44] jtv, I was just looking at it [19:44] jtv, isn't it three in the morning for you right now? [19:49] mars: just about :) === Ursinha is now known as Ursinha-brb === matsubara is now known as matsubara-afk === salgado is now known as salgado-afk === mars changed the topic of #launchpad-reviews to: On call: - || Reviewing: - || queue: [jtv, benji] || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === mars 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 [23:31] https://code.launchpad.net/~thumper/launchpad/recipe-builds/+merge/40489 anyone?