| bac | Meeting Ping: thumper, rockstar, stub, mwhudson, stevek, lifeless, wgrant | 00:03 |
|---|---|---|
| thumper | bac: pong | 00:03 |
| 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:04 |
| wallyworld | thumper: hi | 00:05 |
| bac | hi wallyworld | 00:05 |
| thumper | wallyworld: just thought you might like to sit in | 00:05 |
| 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:06 |
| 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:07 |
| 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:08 |
| 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:09 |
| 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:10 |
| 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:11 |
| bac | so except for our newest team member that means all canonical lp team members are now reviewers | 00:12 |
| bac | (hi wallyworld) | 00:12 |
| 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:13 |
| 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:14 |
| 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:15 |
| 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:16 |
| abentley | bac: Yeah. Well, any arbitrary line, at least. | 00:17 |
| 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:18 |
| 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:19 |
| 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:20 |
| 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:22 |
| bac | thanks for coming. | 00:23 |
| thumper | thanks | 00:24 |
| mwhudson | oops, missed that | 01:24 |
| === 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 | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!