[00:03] Meeting Ping: thumper, rockstar, stub, mwhudson, stevek, lifeless, wgrant [00:03] bac: pong [00:04] me [00:04] bac: lifeless is in the US [00:04] wallyworld: ping [00:04] hey thumper. and wgrant [00:04] bac: and rockstar is elsewhere [00:04] And hopefully StevenK is as well. [00:05] thumper: hi [00:05] hi wallyworld [00:05] wallyworld: just thought you might like to sit in [00:06] thumper: stevenk should be on-line by now, no? [00:06] should be [00:06] thumper: ack [00:06] well, let's start [00:07] the AmEu meeting was pretty uneventful. despite the late warning about the time change most everyone was available. [00:07] bac: Isn't Steve off this week? [00:07] wgrant: perhaps. i think i missed that email from sarah... [00:08] salgado is still a UI reviewer mentat. please consider throwing your UI reviews to him if possible. i think he should be wrapping up soon... [00:09] thumper: how has mentoring StevenK been going? is he getting any more reviews? this process is dragging on for him. [00:09] bac: not mentored anything for a week [00:10] thumper: ok. i've encouraged the ameu folks to give him a shot at reviews but it isn't very practical. [00:10] related, benji is not being mentored by edwingrubbs [00:11] and jcsackett starts as a reviewer this week mentored by sinzui [00:11] I think you mean "now" and not "not". [00:11] sorry EdwinGrubbs, typo [00:12] so except for our newest team member that means all canonical lp team members are now reviewers [00:12] (hi wallyworld) [00:13] outstanding stuff: [00:13] Sinzui to investigate making lint check for the Storm 'in' gotcha [00:13] he has made some progress but doesn't have a fix yet. he does report that his pocket-lint program has been accepted into natty, which is cool. [00:14] * bac to work with mars regarding a slow introduction of new reviewer points regarding the ArchitectureGuide. [00:14] fail [00:14] :( [00:14] I think the final fix is to convert the false SQLObject SQLs to Storm and declare victory [00:14] hi bac. sorry, stepping in and out. getting a cowboy done [00:15] jtv started a discussion about seeing more blatantly bad code in devel and was concerned. [00:15] it was suggested by sinzui that some automated changes post-review can cause style problems [00:16] abentley commented that he is working on a tool (bzr plugin i assume) that will trace every line back to the merge proposal. [00:17] bac: Yeah. Well, any arbitrary line, at least. [00:18] i offered to contact reviewers if other find things that slipped by that should not have. we don't want to do a lot of finger pointing but instead just inform people that they may have missed something. [00:19] personally i'd like a gentle reminder if i did a bad review. hopefully this feedback will be rare and well received. [00:19] and that is what we covered in AmEu. [00:19] anyone have comments or other topics to discuss? [00:19] I think different reviewers apply different levels of pedantry. [00:20] wgrant: the examples cited by jtv weren't really pedantry but things that were just clearly out of place [00:20] Ah, i see. [00:22] and with some of the automated tools we have i feel comfortable deferring a lot of stuff and mention "your imports look funny, run henning's tool before you commit." that plus running lint covers a lot of the drudgery. [00:22] anything else? [00:23] thanks for coming. [00:24] thanks [01:24] oops, missed that === Ursinha-bbl is now known as Ursinha-afk === salgado is now known as salgado-lunch === salgado-lunch is now known as salgado === jelmer is now known as Guest52674 === Guest52674 is now known as jelmer === jelmer is now known as Guest95477 === Ursinha is now known as Ursinha-afk === salgado is now known as salgado-afk