/srv/irclogs.ubuntu.com/2009/12/02/#launchpad-reviews.txt

=== noodles775 changed the topic of #launchpad-reviews to: on-call: - || reviewing: - || queue [sinzui, noodles775] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
=== salgado is now known as salgado-afk
henningenoodles775: Hi! could you do a ui review for bug 425583?10:17
mupBug #425583: Improve language set index page <post-3-ui-cleanup> <Launchpad Translations:Triaged> <https://launchpad.net/bugs/425583>10:17
henningenoodles775: the bug has a mockup attached to it, there is no branch or mp yet.10:18
noodles775henninge: sure.10:18
henningeSeeing how this release week is a quiet one for you ... ;-)10:18
noodles775henninge: I don't suppose I can do a swap with you?10:19
noodles775https://code.edge.launchpad.net/~michael.nelson/launchpad/487009-db-more-soyuz-extraction/+merge/1553310:19
henningenoodles775: sure you can ;)10:19
noodles775Thanks!10:20
noodles775henninge: ok, done. It looks great! I added a few thoughts that came up as I thought about interacting with it.10:53
henningenoodles775: cool, I'll look at it in a minute10:54
henningenoodles775: about your branch10:54
henningewithout wanting to check each line, I trust that the code moved to dispatchBuildToSlave, verifyBuildRequest, _cachePrivateSourceOnSlave, _extraBuildArgs was copied verbatim?10:55
noodles775Almost verbatim, I had to check all references to self, so that self.url would become self._builder.url.10:55
henningenoodles775: and you had to add a call to _extraBuildArgs11:00
noodles775henninge: The extra build arguments used to be created in startBuild(), so yes, I copied that code out into BinaryPackageBuildBehavior._extraBuildArgs() and called it from the specific dispatchBuildToSlave instead.11:02
henningenoodles775: I am trying to decide if this is a pure refactoring branch or if it adds functionality that would need tests.11:04
henningenoodles775: what about the new code for IdleBuildBehavior?11:04
=== matsubara-afk is now known as matsubara
noodles775henninge: yep, as mentioned in the MP, I'm going to add some documentation for the build behavior infrastructure in the next branch, but I've certainly tried to ensure that I'm not adding any new functionality to IBuilder in this branch, just pure refactoring.11:06
henningenoodles775: also the return value of "status" is not demonstrated/tested.11:06
henningenoodles775: ah, I see11:07
=== salgado-afk is now known as salgado
noodles775henninge: so, the current behavior of the status property is tested in doc/builder.txt (not exhaustively), and I've tried to ensure that I haven't modified its behaviour.11:10
noodles775But without a doubt, it could be better tested.11:10
henningenoodles775: oh, I missed that that is moved code, too. Found it now at the end of the patch ;)11:17
noodles775Ah, great :D11:18
noodles775Note, in that case it's split code, rather than moved.11:18
henningenoodles775: let me just verify that the test is really passing and then you're good to go11:18
noodles775Ta!11:18
henningeyes, the 'Idle' went to the Idle...Behavior11:19
henningenoodles775: done r=me11:27
noodles775Thanks henninge !11:27
salgadoallenap, do you plan to write a windmill test for your fix to bug 489342?11:28
mupBug #489342: "'display_name' is undefined" JavaScript error on development <javascript> <Launchpad Bugs:In Progress by allenap> <https://launchpad.net/bugs/489342>11:28
allenapsalgado: Maybe, but I don't really know where to start right now, and I want to get it landed ;)11:28
allenapsalgado: I can promise to write a test and do manual QA for the release.11:29
allenapIf that's okay with you.11:29
salgadoallenap, that's ok, yes, but can you file a bug for the missing test so that we don't forget about it?11:29
=== noodles775 changed the topic of #launchpad-reviews to: on-call: - || reviewing: - || queue [sinzui] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
allenapsalgado: Certainly. Thanks.11:30
allenapsalgado: I'll go land now.11:30
salgadoallenap, please land on db-devel11:30
allenapsalgado: Sure.11:30
salgadoI need to get devel closed11:30
allenapsalgado: Gah! I just submitted it and it's gone in for devel.11:32
allenapsalgado: I can resubmit to db-devel if you want to close devel.11:33
salgadoallenap, delete the branch from LP11:33
allenapsalgado: Okay.11:34
salgadoor push --overwrite a 'devel' branch on top of it11:34
salgadoand then resubmit11:34
salgadoseems to have worked11:35
henningenoodles775: thanks for your comment on the languages page. I just replied to it.11:40
=== henninge is now known as henninge-afk
=== allenap changed the topic of #launchpad-reviews to: on-call: allenap || reviewing: - || queue [sinzui] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
=== allenap changed the topic of #launchpad-reviews to: on-call: allenap || reviewing: sinzui || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
=== henninge-afk is now known as henninge
salgadoallenap, stub's got a RC branch up for review, can you take it?12:37
salgadohttps://code.edge.launchpad.net/~stub/launchpad/opstats-nodb/+merge/1554312:37
marsallenap, ping, would you be able to review a brief set of fixes to the JS unit test suite?  https://code.launchpad.net/~mars/launchpad/fix-js-unittests/+merge/1554612:49
=== henninge is now known as henninge-afk
allenapsalgado: Sure13:34
allenapmars: Sure.13:34
allenapsalgado, stub: r=me13:46
=== allenap changed the topic of #launchpad-reviews to: on-call: allenap || reviewing: sinzui || queue [mars] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
=== henninge-afk is now known as henninge
=== matsubara is now known as matsubara-lunch
=== EdwinGrubbs changed the topic of #launchpad-reviews to: on-call: allenap, Edwin || reviewing: sinzui || queue [mars] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
EdwinGrubbsallenap: would you like me to take mars' branch?15:29
allenapEdwinGrubbs: I did it already, sorry.15:29
=== allenap changed the topic of #launchpad-reviews to: on-call: allenap, Edwin || reviewing: - || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
EdwinGrubbsyou shouldn't be sorry. I should be thanking you for saving me work.15:30
marslol15:30
marsallenap, thanks for the review15:30
allenapHeh :)15:30
allenapmars: Welcome :)15:31
=== beuno is now known as beuno-lunch
=== matsubara-lunch is now known as matsubara
deryckallenap or EdwinGrubbs -- I have a simple js unit test fix.  30 line diff.16:47
=== deryck changed the topic of #launchpad-reviews to: on-call: allenap, Edwin || reviewing: - || queue [deryck] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
deryckhttps://code.edge.launchpad.net/~deryck/launchpad/fix-me-too-unit-test-491441/+merge/1556116:47
EdwinGrubbsderyck: I can take it.16:47
deryckEdwinGrubbs, cool, thanks.16:48
=== allenap changed the topic of #launchpad-reviews to: on-call: Edwin || reviewing: - || queue [deryck] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
EdwinGrubbsderyck: I assume this is not an RC candidate.16:49
deryckEdwinGrubbs, no, it's not.16:49
deryckJust want it ready when pqm opens is all.16:49
derycksimple fix, so catch it while I can.16:49
=== beuno-lunch is now known as beuno
=== EdwinGrubbs changed the topic of #launchpad-reviews to: on-call: Edwin || reviewing: - || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
gary_posterEdwinGrubbs: you have time for a testfix/reroll candidate?  https://code.edge.launchpad.net/~gary/launchpad/opstats-nodb/+merge/1556919:42
EdwinGrubbsgary_poster: sure, looking now19:44
EdwinGrubbsgary_poster: r=me19:47
gary_posterthank you19:50
=== salgado is now known as salgado-afk
=== matsubara is now known as matsubara-afk
=== jamalta_ is now known as jamalta

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