[00:03] <bac> Meeting Ping: thumper, rockstar, stub, mwhudson, stevek, lifeless, wgrant
[00:03] <thumper> bac: pong
[00:04] <wgrant> me
[00:04] <thumper> bac: lifeless is in the US
[00:04] <thumper> wallyworld: ping
[00:04] <bac> hey thumper.  and wgrant
[00:04] <thumper> bac: and rockstar is elsewhere
[00:04] <wgrant> And hopefully StevenK is as well.
[00:05] <wallyworld> thumper: hi
[00:05] <bac> hi wallyworld
[00:05] <thumper> wallyworld: just thought you might like to sit in
[00:06] <bac> thumper: stevenk should be on-line by now, no?
[00:06] <thumper> should be
[00:06] <wallyworld> thumper:  ack
[00:06] <bac> well, let's start
[00:07] <bac> the AmEu meeting was pretty uneventful.  despite the late warning about the time change most everyone was available.
[00:07] <wgrant> bac: Isn't Steve off this week?
[00:07] <bac> wgrant: perhaps.  i think i missed that email from sarah...
[00:08] <bac> 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] <bac> thumper: how has mentoring StevenK been going?  is he getting any more reviews?  this process is dragging on for him.
[00:09] <thumper> bac: not mentored anything for a week
[00:10] <bac> thumper: ok.  i've encouraged the ameu folks to give him a shot at reviews but it isn't very practical.
[00:10] <bac> related, benji is not being mentored by edwingrubbs
[00:11] <bac> and jcsackett starts as a reviewer this week mentored by sinzui
[00:11] <EdwinGrubbs> I think you mean "now" and not "not".
[00:11] <bac> sorry EdwinGrubbs, typo
[00:12] <bac> so except for our newest team member that means all canonical lp team members are now reviewers
[00:12] <bac> (hi wallyworld)
[00:13] <bac> outstanding stuff:
[00:13] <bac> Sinzui to investigate making lint check for the Storm 'in' gotcha
[00:13] <bac> 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> * bac to work with mars regarding a slow introduction of new reviewer points regarding the ArchitectureGuide.
[00:14] <bac> fail
[00:14] <bac> :(
[00:14] <sinzui> I think the final fix is to convert the false SQLObject SQLs to Storm and declare victory
[00:14] <wallyworld> hi bac. sorry, stepping in and out. getting a cowboy done
[00:15] <bac> jtv started a discussion about seeing more blatantly bad code in devel and was concerned.
[00:15] <bac> it was suggested by sinzui that some automated changes post-review can cause style problems
[00:16] <bac> 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] <abentley> bac: Yeah.  Well, any arbitrary line, at least.
[00:18] <bac> 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] <bac> personally i'd like a gentle reminder if i did a bad review.  hopefully this feedback will be rare and well received.
[00:19] <bac> and that is what we covered in AmEu.
[00:19] <bac> anyone have comments or other topics to discuss?
[00:19] <wgrant> I think different reviewers apply different levels of pedantry.
[00:20] <bac> wgrant: the examples cited by jtv weren't really pedantry but things that were just clearly out of place
[00:20] <wgrant> Ah, i see.
[00:22] <bac> 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] <bac> anything else?
[00:23] <bac> thanks for coming.
[00:24] <thumper> thanks
[01:24] <mwhudson> oops, missed that