/srv/irclogs.ubuntu.com/2008/09/23/#launchpad-meeting.txt

=== ursula_ is now known as Ursinha
barry_#startmeeting02:59
MootBotMeeting started at 20:59. The chair is barry_.02:59
MootBotCommands Available: [TOPIC], [IDEA], [ACTION], [AGREED], [LINK], [VOTE]02:59
barry_mootbot!02:59
barry_hello everybody and welcome to this week's asiapac reviewers meeting02:59
barry_who's here today?02:59
barry_anybody?03:00
barry_jml, mwhudson, thumper ping?03:01
jmlhi03:01
thumperbarry_: hi03:01
jmlI forgot it was Tuesday :)03:01
* thumper is somewhat busy with beuno03:01
barry_jml: it's not, it's still monday :)03:02
mwhudsonhi03:02
jmlmy mistake03:02
barry_thumper: i can wait a little bit if you want03:02
thumperbarry_: yeah, but you can't wait for a week :)03:02
thumperI'll just pop in and out03:02
barry_:)03:02
barry_cool.  btw, i will not be here next week.  do you want to skip the meeting or can one of you run it?03:03
barry_[TOPIC] agenda03:03
MootBotNew Topic:  agenda03:03
barry_ * Roll call03:03
barry_ * Using ReST instead Moin for documentation -- flacoste  [<<Date(2008-09-22T10:17:00-0500)>>]03:03
barry_ * Naming conventions for unit test methods. `testFooBar`, `test_fooBar` and `test_foo_bar` all exist. Recommend settling on `testFooBar` and only changing existing ones as encountered in normal work. -- jml [<<Date(2008-09-10T13:48:09+1000)>>]03:03
barry_ * Reviewers remove requests from Pending Reviews when you start a review.  If you forget the next on-call reviewer may duplicate your work.  -- bac [<<Date(2008-09-16T10:07:09-0500)>>]03:03
barry_ * If there's time, the old boring script03:03
barry_   * Next meeting03:03
barry_   * Action items03:03
barry_   * Queue status03:03
barry_   * Mentoring update03:03
* jml waits03:04
barry_i'll let you guys sort it out for next week then :)03:05
barry_[TOPIC]  * Using ReST instead Moin for documentation -- flacoste  [<<Date(2008-09-22T10:17:00-0500)>>]03:05
MootBotNew Topic:   * Using ReST instead Moin for documentation -- flacoste  [<<Date(2008-09-22T10:17:00-0500)>>]03:05
thumperI really don't care about this one03:05
barry_i haven't talked to flacoste about this yet, but i think this was brought up by gary03:05
barry_gary wants to write doctests in reST03:06
* barry_ likes reST03:06
jmlbarry_: given that we aren't actually publishing the documentation, I don't see that it matters too much.03:06
mwhudsonunless we're actually going to render the documentation and do something with it, i heartily don't care03:06
barry_cool.  i just wanted to get your sense so i can communicate it accurately at the ameu meeting03:07
mwhudson(rendering the documentation would probably be a good idea though)03:07
barry_sounds like a resounding -0/+003:07
jmlbarry_: can we change the doc standard so it's 78 cols all the way through?03:07
mwhudsonbarry_: well, it's a bit more negative than that, as it would be a lot of work03:07
* jml is half kidding03:07
barry_jml: i've been wanting to do that since i started03:07
mwhudson.oO(not that i edit doctests much...)03:08
jmlbarry_: 72 for text and 78 for Python seems like make-work to me.03:08
barry_mwhudson: i wouldn't propose to rewrite existing docs just to reformat them.  do it as general cleanup03:08
jmlbarry_: but that's probably a separate agendum :)03:08
barry_jml: why do you think i avoid friday reviews?  sinzui is the only one who cares about that :)03:08
barry_jml: but now sinzui is team leader and not doing reviews, so there ya go :)03:09
jmlheh heh03:09
mwhudsonbarry_: still more work than seems worth it unless there are other benefits03:09
barry_jml: i will put it on the agenda for wednesday03:09
barry_[ACTION] barry put 78 column consistency in doctests on agenda for ameu03:09
MootBotACTION received:  barry put 78 column consistency in doctests on agenda for ameu03:09
barry_mwhudson: ok, thanks.  gotcha03:10
jmlcool.03:10
* jml agrees with mwhudson03:10
barry_[TOPIC]  * Naming conventions for unit test methods. `testFooBar`, `test_fooBar` and `test_foo_bar` all exist. Recommend settling on `testFooBar` and only changing existing ones as encountered in normal work. -- jml [<<Date(2008-09-10T13:48:09+1000)>>]03:10
MootBotNew Topic:   * Naming conventions for unit test methods. `testFooBar`, `test_fooBar` and `test_foo_bar` all exist. Recommend settling on `testFooBar` and only changing existing ones as encountered in normal work. -- jml [<<Date(2008-09-10T13:48:09+1000)>>]03:10
jmlI had a look at the AMEU minutes.03:10
barry_jml: we talked about this in ameu, but you hadn't had time to state your position yet03:10
jmlit seems they settled on test_<identifier>_qualifying_comments03:10
barry_jml: right, are you okay with that? or disagree?03:11
jmlI assume that tests without identifiers just get pep8'd?03:11
barry_yep03:11
jmlbarry_: I'm cool with it.03:12
barry_jml: cool03:12
jmlbarry_: I basically just wanted a standard -- every module I use is different.03:12
barry_jml: right, agreed!03:12
mwhudsonhooray03:13
barry_[TOPIC]  * Reviewers remove requests from Pending Reviews when you start a review.  If you forget the next on-call reviewer may duplicate your work.  -- bac [<<Date(2008-09-16T10:07:09-0500)>>]03:13
MootBotNew Topic:   * Reviewers remove requests from Pending Reviews when you start a review.  If you forget the next on-call reviewer may duplicate your work.  -- bac [<<Date(2008-09-16T10:07:09-0500)>>]03:13
barry_i think this one's just a heads up.  several people did multiple reviews03:14
jmloh that's right, we still have a wiki page :)03:14
mwhudson"what is this Pending Reviews you speak of" ?03:14
barry_:)03:14
mwhudsoni guess it's worth making the point03:14
* barry_ looks to thumper and beuno03:14
jmlbarry_: incidentally... https://code.edge.launchpad.net/~launchpad/launchpad/trunk03:14
thumpersure03:15
thumperjml: shhh03:15
thumper;-)03:15
* jml needs to read over that "actually using Launchpad for reviews doesn't work" email03:15
* thumper too03:16
barry_jml: whoa03:16
thumperbarry_: FYI, beuno and I are making reviews rock in LP03:17
barry_i can has branches on lp?03:17
barry_thumper, beuno i am very excited to see the results03:17
jmlon the sly, yes.03:17
beunobarry_, it's very promising  :)03:17
jmlbarry_: although make sure you upgrade everything (branches and repos) to 1.603:17
beunoit should make the workflow almost natural03:18
barry_jml: cool.  eta for switching to?03:18
thumperbarry_: also use bzr 1.7 or later03:18
barry_beuno: eta?03:18
thumperbarry_: and the scanner will choke somewhat until mwhudson's branch is cherrypicked03:18
mwhudsoni guess we could switch in maybe a week?03:18
jmlbarry_: there are a few patches we need to land.03:18
jmldepending on how the cherry blossoms fall03:19
beunobarry_, it depends on thumper's speed and availability, but we should ahve something submitable after this week03:19
barry_after mwhudson branch is cherrypicked?  if we could do it on or before epic, that would be fantastic03:19
mwhudsonthat should be possible for sure03:19
thumperbarry_: within a day or two for production is my guess03:19
barry_wow. it would be fantastic if we can dogfood all this, kill off devpad branches and PR at epic03:20
* thumper hopes too03:21
* beuno hopes three03:21
thumperI do have two weeks leave between now and the epic03:21
barry_cool.  that's about it from me.  queue update, mentor update, action items, blah blah blah03:23
barry_anything from you guys?  anything you want me to bring up at ameu?03:23
mwhudsonnothing springs to mind03:24
jmlnope03:24
thumpernada03:24
barry_cool.  thanks guys.  see you all in 2 weeks.03:24
barry_#endmeeting03:24
thumperok03:24
MootBotMeeting finished at 21:24.03:24
thumperlater dude03:24
jmlbarry_: see ya03:25
=== bac_afk is now known as bac
=== salgado-afk is now known as salgado
=== bac is now known as bac_breakfast
=== mrevell is now known as mrevell-lunch
=== bac_breakfast is now known as bac
=== bac is now known as bac_standup
=== bac_standup is now known as bac
=== ursula_ is now known as Ursinha
=== mrevell-lunch is now known as mrevell
=== ursula_ is now known as Ursinha
=== bac is now known as bac_lunch
=== salgado is now known as salgado-lunch
=== bac_lunch is now known as bac
=== salgado-lunch is now known as salgado
=== bac is now known as bac_bbiab
=== bac_bbiab is now known as bac
=== bac is now known as bac_afk
=== thumper_laptop is now known as thumper
=== salgado is now known as salgado-afk

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