/srv/irclogs.ubuntu.com/2010/10/20/#launchpad-meeting.txt

=== Ursinha-afk is now known as Ursinha
=== Ursinha is now known as Ursinha-afk
=== Ursinha-afk is now known as Ursinha
rockstar#startmeeting15:00
MootBotrockstar, There is already a meeting in progress.15:00
rockstarWhy is there already a meeting in progress?15:01
rockstar#endmeeting15:01
rockstar#startmeeting15:01
MootBotrockstar, There is already a meeting in progress.15:01
gary_posterheh15:01
rockstarAlright, so MootBot sucks.15:01
sinzuiI think someone need to pull HAL's memory15:01
rockstarHi everyone.  I'm chairing today's reviewer's meeting since bac is travelling the world this week.15:02
rockstarWho's here?15:02
adeuring1me15:02
gary_posterme15:02
henninge,e15:02
sinzuime15:02
henningeme, even15:02
=== adeuring1 is now known as adeuring
abentleyme15:02
salgadome15:02
rockstar6 people huh?15:03
gary_posterI pinged leonardr and mars15:03
abentleyrockstar: I can pretend to be two people, if you like.15:03
leonardrme15:03
sinzuiEdwinGrubbs, meeting?15:03
allenapme15:03
rockstarabentley, if you were on another team, that might help.  As it stands, you're usually the only code guy here anyway.  :)15:03
gary_poster...and there were eight...15:04
deryckme15:04
flacosteme15:04
not-abentleyme15:04
marsme15:04
gary_posterheh15:04
rockstarAlright, let's not sit and wait for the folks who couldn't remember the meeting that happens at the same time every week.15:04
rockstarHere's the past actions from last week15:05
rockstarSinzui to investigate making lint check for the Storm 'in' gotcha15:05
rockstarHenningE to add a note to the PSG about the use of any15:05
sinzuiNo progress yet15:05
rockstarI suspect bac didn't finish that past action, but that henninge knows what it ends with.15:05
henningeI did that15:05
rockstarbac to work with mars regarding a slow introduction of new reviewer points regarding the ArchitectureGuide. (undone)15:06
jelmerme15:06
rockstarhenninge, hooray! You get a gold star.15:06
henningerockstar: it's about the "any" function ;-)15:06
rockstarhenninge, ah, okay. Well, I guess it was a finished statement.  :)15:06
rockstarOkay, we're 1 for 3 on finishing items.15:07
rockstarMentat report!15:07
rockstarSo, I talked with bac and sinzui about this, but I think it's high time henninge graduates as a UI reviewer.15:07
rockstarsalgado, how's mentat life?15:08
* henninge is excited!15:08
salgadogoing fine so far15:08
rockstarOkay.  I guess that's it for that part of the agenda.15:09
rockstarNew items15:09
rockstarMake all reviewers a UI mentat similar to the way we do javascript reviews. (lifeless)15:09
abentleyrockstar: do not want.15:10
rockstarSo, this is not new.  It has been entirely acceptable to get UI reviews from two non-UI reviewers.  It's been that way since the 3.0 work.15:10
rockstarIt's just a PITA to get wrangle more people to do UI reviews.15:11
rockstarabentley, I am curious why you don't want to be a UI reviewer.15:11
marsrockstar, do, as in, become a reviewer?15:11
jelmerrockstar: As I understand it it's ok to get two UI reviews from UI mentats, not necessarily from non-UI reviewers.15:11
sinzuiIts a PITA to get people who want to become a full UI reviewer15:11
sinzuithere isnt a queue for this15:11
rockstarsinzui, yes, that's true too.15:11
abentleyrockstar: I don't want to go through another training process, I don't want to have to do more reviews.15:12
sinzuiWe supposed that the JS/UI team would all becomes reviewers. Most scared to the four winds when the meeting was abandoned15:12
rockstarjelmer, we've kinda strayed from it, but during the 3.0 work, we needed UI reviews A LOT, and so everyone became a "mentat"15:12
abentleyrockstar: I don't think that I have especially good visual design skills.15:12
rockstarsinzui, they were scaring that way before the meeting was abandoned.15:12
rockstarabentley, and you have no desire to develop good visual design skills?15:13
jelmerrockstar: I didn't know that, thanks.15:13
abentleyrockstar: there are many things more important to me than that.15:13
marssounds fair15:13
sinzuiabentley, While I do not know your skills, I have raised a similar point with beuno. Most of use were hired for code design skills, not UI design15:13
rockstarAnyone else have opinions?  I can bring these thoughts up in the AsiaPac meeting as well.15:13
marsrockstar, the way I do it now is I say "You need a UI review - as rockstar or sinzui"15:14
marsrockstar, will that process change?15:14
marshow will people's expectations for UI reviews change?15:14
rockstarmars, yeah, that's a good question.15:15
marshow will the way they ask for them change, and will they expect something different from their reviewer in return?15:15
rockstarThe 3.0 UI still wasn't consistent.  The mentat/mentee process makes it so that we at least get a *ahem* "trickle down" affect.15:15
marsdifferent from their currently-code-only reviewer15:15
marsrockstar, so you are asking, if everyone became a UI mentee, would quality suffer?15:16
marssince the current process is a self-selecting one15:17
rockstarmars, I'm not asking anything.  I'm "facilitating."15:17
gary_poster:-)15:17
sinzuimars, 3.0 did suffer by the 2-person-yes rule15:17
sinzuiWe landed dozens of pages without breadcrumbs and we did not discover that until weeks later15:18
sinzuiI favour 2-person approval, but I think we still need some review to ensure we are not codifying convention as rules15:19
marsrockstar, do you know what lifeless' goal was in proposing this?15:19
rockstarmars, I think he wants every reviewer to be able to do any review.15:20
rockstarI started to propose a similar javascript review process, but I still don't feel like we, as a team, have a good understanding of javascript, so I backed off.15:21
marsok, so15:21
gary_posterwell, I assume he wants to get a higher throughput on our landing process15:21
gary_posterand guesses that this would help15:21
rockstargary_poster, yeah, exactly.15:21
marsright15:21
rockstarEspecially since there are no antipodean UI reviewers currently.15:21
henningebut will it help our UI quality?15:21
gary_posterah, that's a big deal15:21
sinzuiI do not think every engineer can be a UI reviewer.15:21
marsso it is just a way to do a light check-up, and defer the "qualified" review until later15:22
marsand you can land in the meantime15:22
gary_poster"can be" is a big statement, but I agree with sinzui's sentiment, if not the wording15:22
deryckif not for abilities as much as lack of desire to learn ;)15:23
rockstarAlright, I'll take this discussion to the AsiaPac meeting and see what happens.15:23
gary_posterright deryck :-)15:23
rockstarPeanut gallery, for anyone who wishes to throw peanuts.15:23
* gary_poster eats a peanut instead15:24
henningeAre we on 2.6. now or not?15:24
marsno15:24
gary_posterI'll ask the losas about the pqm machine15:24
gary_posterthat's the last hold out15:24
rockstarAwesome.15:25
rockstarAnything else?15:25
rockstar515:25
rockstar415:25
rockstar315:25
bigjools615:25
rockstar215:25
rockstar115:25
henninge7?15:25
rockstarOkay, that's a wrap.  Thanks for coming everyone!15:25
marsthanks rockstar15:25
henningerockstar: Thank you!15:25
jelmerthanks Paul15:25
gary_posterthank you15:26
=== salgado is now known as salgado-lunch
=== Ursinha is now known as Ursinha-lunch
=== Ursinha-lunch is now known as Ursinha
=== salgado-lunch is now known as salgado
=== gary_poster_ is now known as gary_poster
=== Ursinha is now known as Ursinha-afk
=== Ursinha-afk is now known as Ursinha

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!