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

thumperhttps://code.edge.launchpad.net/~thumper/launchpad/lp-code-errors/+merge/1572402:16
thumperand now https://code.edge.launchpad.net/~thumper/launchpad/claim-review-into-model-attempt2/+merge/1572804:06
adiroibanhenninge: hi. is the submission now open?07:37
=== henninge changed the topic of #launchpad-reviews to: on-call: henninge || reviewing: - || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
henningeadiroiban: yes, channel topic on #lp-dev says "PQM is open for srs bsns" ;-)07:39
henningeGood morning adiroiban :)07:39
adiroibansrs bsns ? Serious Business ?07:40
henningecorrect07:40
adiroibandoes it mean any kind of MP ?07:41
henningeour losas, aren't they just a funny bunch ? ;-)07:41
henningeadiroiban: yes, the other states are "closed", which was last week, and "testfix" which may happen anytime when the test suite breaks on the buildbot.07:42
adiroibanhenninge: I see. That make sense now :) 07:43
adiroibanhenninge: will you "land" my branch , or is there something I need to do?07:46
henningeadiroiban: hang on, let me check on the situation07:49
henningeadiroiban: please merge the current devel into your branch and push it again.07:54
henningeadiroiban: like this: http://paste.ubuntu.com/336365/07:56
adiroibanhenninge: much appreciated :)07:57
henningewatch for merge conflicts07:57
adiroibanhenninge: and I should create a new MP ?08:05
henningeadiroiban: no08:05
henningeadiroiban: if you didn't have any serious merge conflicts, we can just land it.08:06
henningeadiroiban: what was the branch again?08:06
adiroibanhttps://code.edge.launchpad.net/~adiroiban/launchpad/bug-97293/+merge/1539108:06
adiroibanthere were only 2 lines of code :)08:06
adiroibanfrom bzr merge -> All changes applied successfully.08:07
henningecool08:09
adiroibanbranch was pushed08:09
henningeadiroiban: a, I remember08:09
henningeadiroiban: yup got. Ok, nothing else for you to do now, I will land it.08:10
adiroibanhenninge: thanks. An for the other one I should do the same merge and ping the person who told me that will land the branch?08:11
henningeadiroiban: just ping me08:11
henningeafter the pushing08:11
adiroibanok08:11
adiroibanhenninge: https://code.edge.launchpad.net/~adiroiban/launchpad/bug-193750 merged...08:15
adiroibanshould I give you the full link to the branch  or to the MP, or just the MP number?08:15
henningeadiroiban: all good, I got it.08:17
henninge;)08:17
adiroibanhenninge: :) ... another one https://code.edge.launchpad.net/~adiroiban/launchpad/bug-487970/+merge/1525008:18
adiroibanhenninge: last one: https://code.edge.launchpad.net/~adiroiban/launchpad/bug-135008/+merge/1534708:26
henningeadeuring: wow, you have been busy! Great job!08:27
henningeadiroiban: ah, one more thing you can do: Add a commit message to each mp!08:30
henningethat way I don't have to make them up  ... ;)08:31
adiroibanhenninge: hm... how?08:31
adiroiban:)08:31
henningeadiroiban: simply click on "Set commit message" :)08:31
henningeIt's a new feature!08:31
henningeadiroiban: It's a green line starting with a (+) icon.08:32
adiroibanAh... it was just my nose covering that link :)08:32
henningelol08:32
adiroibangot it08:33
adiroibanand what should I write there?08:33
henningeit's just a bzr commit message to use when your  branch is merged and commit into devel.08:34
henningeso you can probably just copy something you used for your local commits.08:35
adiroibanyes. thanks08:35
henninge"copying plural values will also enable the required fields"08:35
henningeadiroiban: But make it proper sentences, please. ;)08:35
henninge"Copying plural values will also enable the required fields."08:35
adiroibanhenninge: done for all 4 MPs08:46
henningeadiroiban: cool, thanks. You are, done. I'll keep you posted on the proress during the day.08:46
adiroibanhenninge: thanks. I will also need a pre-MP review for https://code.edge.launchpad.net/~adiroiban/launchpad/bug-43124908:48
adiroibanshould I ask it on lp-reviews or via the bug tracker?08:49
henningeadiroiban: no, using the bug tracker for code reviews is not what we do.08:51
henningeadiroiban: asking here or directly one of the lp developers is what you should do.08:51
adiroibanhenninge: here or on lp-dev ?08:52
henningeadiroiban: Incidently I am an on-call reviewer (OCR) today.08:52
henninge;)08:52
henningeadiroiban: "here" is #lp-reviews08:52
henningebut I am also Help contact and currently busy landing some great contributions submitted by a cool community member ... ;-)08:53
adiroibanOCRs are also doing pre-MPs reviews ?08:54
adiroibansorry for all these questions... but I'm lost in this process08:54
henningeadiroiban: first one's in! http://bazaar.launchpad.net/~launchpad-pqm/launchpad/devel/revision/997709:24
henningeadiroiban: the others will take longer, though. I did not run this through the test suite since it does not yet run javascript tests.09:24
henningeI just ran the tests locally and submitted directly.09:25
adiroibanhenninge: ok. np. thanks09:25
henningeadiroiban: was this your first contribution to Launchpad?09:30
adiroibanhenninge: yep. 2 lines ... like 0.02$ :)09:32
henningeadiroiban: Well, Congratulations!09:33
henninge;-)09:33
danilosadiroiban, henninge: hi :)10:16
adiroibandanilos: hi :D10:16
henningehi danilos!10:16
adiroibandanilos: I would need a quick review for this branch so see if I should push it to a MP https://code.edge.launchpad.net/~adiroiban/launchpad/bug-43124910:17
danilosadiroiban, I've seen you looked into a gazillion of bugs :) I'll be going through them and commenting; in general, I suggest you talk about the bugs beforehand (i.e. before starting on them), just to make sure you are not wasting your time if we disagree what the solution should be10:18
danilosadiroiban, sure, looking at bug 43124910:18
mupBug #431249: Put navigation links in a seperate template/macro. <post-3-ui-cleanup> <Launchpad Translations:In Progress by adiroiban> <https://launchpad.net/bugs/431249>10:18
adiroibandanilos: i agree... but the goal of all those bugs was to get into the code. Even if that work will not be used, I still learnt how to hack LP10:20
danilosadiroiban, sure :)10:20
danilosadiroiban, I am not saying it's not great, just giving an advance notice :) it's possible they are all doing exactly as desired :)10:20
adiroibandanilos: so you can consider this is the pre implementation discussion. I was aware of this risk10:21
adiroibanthere is no problem if that code will not be used... as you can see, I start working on some bugs that were not even confirmed10:21
danilosadiroiban, right, bug 431249 fix looks reasonable; I'd still keep the page ID something like "translation-macros", even though it's registered only on TranslationsLayer, because this might hide a generic +macros if it exists10:22
mupBug #431249: Put navigation links in a seperate template/macro. <tech-debt> <ui> <Launchpad Translations:In Progress by adiroiban> <https://launchpad.net/bugs/431249>10:22
danilos(or if it's created in the future)10:22
danilosadiroiban, otherwise, it looks like a good approach, so, provided there are sufficient tests, go for it :)10:22
adiroibandanilos: ok. I can change that10:22
adiroibandanilos: should I add more test ? 10:23
danilosadiroiban, not necessarily, only add a test if there's not any one testing the generic presence of these links on both pages - do not go through all the conditional tests etc.10:23
adiroibanthose changes should not touch any „visible” part10:23
adiroibanaha. ok10:24
danilosadiroiban, right, so, it's two options: find existing test or write one :) former is usually simpler :)10:24
danilosadiroiban, btw, did you get help from Henning about landing your already approved branches?10:24
adiroibandanilos: yep. everhing is ok.10:25
adiroibandanilos: the other feedback I'm asking is about this one: https://code.edge.launchpad.net/~adiroiban/launchpad/bug-40647710:26
danilosadiroiban, looking10:29
danilosadiroiban, in general, it's fine; however, I'd rather structure those permissions differently, and make use of launchpad.TranslationsAdmin permission for this role instead of using launchpad.Admin (though, note that you'd have to change .zcml in that case as well, perhaps even some templates something/required:launchpad.Admin conditions)10:32
danilosadiroiban, but, if you want to keep it simple, just go with what you've got (what I am suggesting is simply because the current state of privileges/permissions in LP is a bit of a mess, and I'd like to clean it up :)10:33
adiroibandanilos: I can look into that. This was my first look at LP privilege system, and I just wanted to make sure I got it right.10:34
adiroibandanilos: thanks. This should could keep me bussy for some time :)10:37
danilosadiroiban, heh, nah, thank you :)10:39
=== matsubara-afk is now known as matsubara
=== mrevell is now known as mrevell-lunch
salgadohenninge, trivial one for you: http://paste.ubuntu.com/336479/11:55
henningesalgado: what is that file for?11:56
* henninge is reading the file ... ;)11:56
salgadoyeah, the heading there says it all11:56
henningesalgado: so, I wonder why this was able to land, then?11:57
henningesalgado: r=me, of course ;)11:57
salgadohenninge, because PQM doesn't run the testsuite?11:57
henningesalgado: ;) I see it now ...11:58
henningesalgado: did you see my r=me ?12:00
salgadohenninge, yep, thanks!12:00
henningeadeuring: Hi!12:02
adeuringhi henninge12:02
henningeadeuring: Did you land a branch for Adi this morning?12:02
adeuringhenninge: yes12:03
adeuringlp:bug-19275012:03
henningeadeuring: ok, I have the same branch in ec2 for him. I think he wasn't aware that you were going to do that, too.12:03
adeuringhenninge: then you can kill the ec2 instance, I think12:04
henningeadeuring: oh yeah, np with that12:04
henninge I was just wondering what happened12:04
henningeadeuring: it's bug 19-3-750, though. 12:06
mupBug #19: "Display settings" forgotten when "Save" is pressed <Launchpad Translations:Fix Released by daf> <https://launchpad.net/bugs/19>12:06
henningemup: bug 19375012:06
mupBug #193750: remove references to +addticket <trivial> <ui> <Launchpad Translations:In Progress by adiroiban> <https://launchpad.net/bugs/193750>12:06
adeuringhenninge: yes12:06
salgadohenninge, I have yet another trivial one for you: http://paste.ubuntu.com/336490/12:27
salgadothis one reverts a couple changes I committed to mainline by accident12:27
henningesalgado: I guess it's too early in the morning for you! ;)12:29
henninges/is/was/12:29
salgadoyeah, today didn't start very well12:30
henningesalgado: shouldn't this patch be the other way round?12:32
henningeI just compared it to the patch you want to revert.12:32
salgadohenninge, yes, the other way around is what I'm committing, sorry12:32
salgadoI applied it with -R, but gave you the original diff, where the change was introduced12:33
salgadohttp://paste.ubuntu.com/336495/ is the correct one12:33
henningesalgado: cool, r=me12:33
=== henninge is now known as henninge-lunch
salgadothanks henninge-lunch 12:34
=== henninge-lunch is now known as henninge
=== mrevell-lunch is now known as mrevell
henningeHi mrevell !13:14
mrevellhi henninge13:14
henningemrevell: Do you know what help@lp.net points to?13:14
mrevellhenninge, yes, it points to feedback@launchpad.net13:15
henningemrevell: sure? I got an email this morning addressed to me an to help but I only saw one copy.13:16
henningeI am subscribed to feedback.13:17
mrevellhenninge, let me checl13:17
mrevells/checl/check13:17
henningemrevell: nm, I checked, too. The other one'sin the bin ...13:19
mrevellhenninge, I don't see anything in the queue13:19
henningesorry13:19
henningeargh!13:20
henningeno, that's not it13:20
=== sinzui changed the topic of #launchpad-reviews to: on-call: (henninge, also on CHR) || reviewing: - || queue [sinzui, sinzui] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
=== abentley changed the topic of #launchpad-reviews to: on-call: (henninge, also on CHR), abentley || reviewing: - || queue [sinzui, sinzui] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
EdwinGrubbsabentley: can you review https://code.edge.launchpad.net/~edwin-grubbs/launchpad/bug-433809-picker-search-slash/+merge/1567415:23
abentleyEdwinGrubbs: Okay15:23
=== abentley changed the topic of #launchpad-reviews to: on-call: (henninge, also on CHR), abentley || reviewing: -, EdwinGrubbs || queue [sinzui, sinzui] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
noodles775henninge or abentley: could either of you review an incremental to an MP which was approved last cycle (but not landed?). See my last comment at:15:24
noodles775https://code.edge.launchpad.net/~michael.nelson/launchpad/443353-api-builds-from-private/+merge/1489615:24
=== henninge changed the topic of #launchpad-reviews to: on-call: abentley || reviewing: EdwinGrubbs || queue [sinzui, sinzui] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
henningenoodles775, abentley: Sorry, got my hands full on CHR.15:24
noodles775np, it's not urgent.15:24
abentleynoodles775: I'll queue it.15:25
noodles775Thanks abentley 15:25
=== abentley changed the topic of #launchpad-reviews to: on-call: abentley || reviewing: EdwinGrubbs || queue [sinzui, sinzui, noodles775] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
abentleyEdwinGrubbs: In the future, you can set db-devel as the prerequisite branch and still propose merging into devel.15:26
noodles775oh, abentley, on that note, I've got a pipeline of MPs that were approved, but are all proposed for db-devel (as they were dependent on schema changes). I'm keen to land them now in devel. What would you recommend as the best way?15:28
noodles775(I'm about to just aggregate all the changes into a new devel branch, propose it as an MP and link to the previous approvals before ec2 land'ing it).15:29
abentleynoodles775: Personally, I would land the last pipe on devel using pqm-submit.15:29
abentleynoodles775: I'm not sure whether ec2 land will allow you to override the target.  I would assume not.15:30
noodles775abentley: ok, I'll try that. Thanks!15:30
noodles775(once I've QA'd it on dogfood etc.)15:30
abentleynoodles775: No problem.  (this implies you'll need to run the tests if you haven't already)15:31
noodles775abentley: yes, I'll re-merge db-devel, run the tests, QA on dogfood, and if all goes well land that branch on devel.15:31
abentleynoodles775: I would recommend merging devel rather than db-devel.15:32
=== matsubara is now known as matsubara-lunch
=== abentley changed the topic of #launchpad-reviews to: on-call: abentley || reviewing: noodles775 || queue [sinzui, sinzui, EdwinGrubbs] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
abentleynoodles775: I'm concerned because this diff includes a lot of changes that don't seem to be in scope, e.g. YUI -> LPS.15:56
noodles775abentley: I'm not sure why you say not in scope? The core change of the branch was to getBuildsForSources() - which affected the UI as outlined in the incremental?15:58
abentleynoodles775: Your incremental diff doesn't show the YUI -> LPS change.15:59
noodles775abentley: LPS?15:59
abentleynoodles775: Look that the preview diff.  It has a bunch of YUI -> LPS changes in it.16:00
noodles775abentley: ah, you mean the generated diff - I'm assuming that's because I've just merged devel and pushed as you suggested (but the MP is for db-devel).16:02
noodles775The actual diff for this change is in the comment.16:03
=== noodles775 is now known as noodles775-otp
abentleynoodles775-otp: I see.  I didn't realize you'd be asking for review on any of those.16:05
noodles775-otpabentley: just the incremental. But yes, after I did that, I thought a better option would have been to add a new pipe and do it there.16:06
abentleynoodles775-otp: So to be clear: you actually want to land this on devel, not db-devel.16:07
noodles775-otpabentley: correct. I had to develop it using db-devel last cycle, but want to land it in devel now.16:08
noodles775-otpIs there a way to re-target the MP at devel?16:08
abentleynoodles775-otp: No, that would be a different proposal, which makes sense because there are different criteria for landing on devel.16:09
noodles775-otpabentley: OK (although I can see there would be cases where it matters, but in this case it's really just the diff being reviewed).  I just thought I had memories of using hitchhiker to do it once before.16:10
abentleynoodles775-otp: No, hitchhiker only controls branches.  It doesn't provide a way to manipulate merge proposals.16:11
noodles775-otpOK. Let me know if I should revert r9893 on that branch then.16:12
abentleynoodles775-otp: Don't worry about it, I'm regenerating the preview diff locally.16:12
noodles775-otpok, thanks.16:13
abentleynoodles775-otp: Are both of the finds necessary, or would the second find (for copied binaries) work even if SourcePackagePublishingHistory.archiveID == Build.archiveID ?16:25
noodles775-otpabentley: hrm, I thought so when I wrote that a few weeks ago, but looking at the two finds now, I'm struggling to see why.16:35
=== noodles775-otp is now known as noodles775
noodles775BTW: If you're willing to re-review the entire MP, I could create a new MP against devel.16:36
abentleynoodles775: Okay, I don't think it's a blocking issue, but if we can simplify by removing the first find, that would be great.  So r=me, do what you will.16:36
noodles775abentley: OK, I'll try it and see. Thanks!16:37
=== matsubara-lunch is now known as matsubara
=== abentley changed the topic of #launchpad-reviews to: on-call: abentley || reviewing: sinzui || queue [sinzui, EdwinGrubbs] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
abentleyEdwinGrubbs: In case you didn't see earlier, I've bumped your review to the queue because you weren't around to answer my questions.16:41
EdwinGrubbsabentley: I was only gone for a few minutes and then you were offline so I couldn't respond.16:42
abentleyEdwinGrubbs: Sorry about that, my IRC client doesn't make that very obvious.16:42
=== abentley changed the topic of #launchpad-reviews to: on-call: abentley || reviewing: EdwinGrubbs || queue [sinzui, sinzui] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
abentleyEdwinGrubbs: Okay, so is the conditional needed?16:43
EdwinGrubbsabentley: do you mean this conditional:     if '/' in query:16:44
abentleyEdwinGrubbs: No, I mean in patch line 100, "if result is not None".16:45
abentleyEdwinGrubbs: It looks like the code should actually read "result = IStore(...).find(...).one(), so that you can raise lookuperror if there were no results.16:46
EdwinGrubbsabentley: well, I assume the original purpose of that is to provide the LookupError exception as opposed to something more generic. A try/except clause could also be used, but that would be less efficient.16:47
abentleyEdwinGrubbs: I thought find could never return None.  Am I wrong?16:47
EdwinGrubbsabentley: oh, I see what you are saying. I didn't think about the differences in the return values between sqlobject and storm. I'll test that I'm testing the right return value for an unsuccesful find().16:49
abentleyEdwinGrubbs: Great.  I'll leave the review alone for now.  Please ping me when you've examined that.16:51
EdwinGrubbsok16:51
=== abentley changed the topic of #launchpad-reviews to: on-call: abentley || reviewing: sinzui || queue [sinzui] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
salgadoabentley, I have a one-liner for a testfix branch; can I skip the queue? http://paste.ubuntu.com/336670/17:09
abentleysalgado: I don't know what this LPS thing is.17:10
salgadoabentley, I replaced all YUI() calls with LPS in our templates17:10
salgadoLPS is just a YUI instance, stored globally to be used in templates17:11
abentleysalgado: very well, r=me.17:11
salgadothanks abentley 17:12
=== abentley changed the topic of #launchpad-reviews to: on-call: abentley || reviewing: - || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
=== abentley changed the topic of #launchpad-reviews to: on-call: abentley (on lunch) || reviewing: - || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
sinzuiabentley: your understanding of translatables not being batched is correct.17:33
adiroibansalgado: I got an email from Buildbot, but I can not access this link  https://lpbuildbot.canonical.com/builders/lp/builds/40517:34
adiroibancan you please copy the part describing the build problem?17:35
salgadoadiroiban, no need to worry about it; you got that because my changes caused a test to fail17:35
salgadoI've already fixed the test and submitted it to pqm17:35
adiroibansalgado: ok. thanks. I was not sure what to do with it :)17:36
danilossalgado, hey, got a time to take a peek at https://code.edge.launchpad.net/~danilo/launchpad/bug-493629/+merge/15758 (it's a one line change which hopefully fixes the JS buildbot for good, patch by mars actually) - I know you are not OCR, but just so I don't have to wait for abentley to come back from lunch and I can go for dinner :)17:44
* salgado checks the diff17:46
salgadodanilos, r=me17:47
danilossalgado, thanks17:48
=== henninge_ is now known as henninge
=== abentley changed the topic of #launchpad-reviews to: on-call: abentley || reviewing: - || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
adiroibanabentley: hi, Can you please review https://code.edge.launchpad.net/~adiroiban/launchpad/bug-431249/+merge/15767 ?19:40
abentleyadiroiban: Okay, I'll have a look in a minute19:40
=== abentley changed the topic of #launchpad-reviews to: on-call: abentley || reviewing: - || queue [adiroban] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
adiroibanabentley: thanks.  no hurry :)19:41
=== abentley changed the topic of #launchpad-reviews to: on-call: abentley || reviewing: adiroban || queue [] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
abentleyadiroiban: Has this branch been through a complete test suite run?20:00
adiroibanabentley: nope. Only the translation tests.20:01
adiroibanon my computer20:01
abentleyadiroiban: It's approved, so let me know when it's been through a complete test suite run, and I'll land it.20:01
adiroibanabentley: should I run the full test on my computer?20:02
abentleyadiroiban: Yes, that's fine.20:02
adiroibanon the previous one Henning launched the test on some supercomputer 20:03
adiroibanrunning all tests on my computer would take many hours... 20:03
abentleyadiroiban: I run all the tests on my computer.20:04
adiroibanabentley: thanks. 20:09
abentleyadiroiban: There's no supercomputer, he would have just used Amazon's Elastic Compute Cloud.20:10
adiroibancan I run the complte test on a remote computer with no X libs? I have a shell account on a remote computer I can configure it for testing my branches20:10
adiroibanbut I'm not sure about testing Javascript20:10
abentleyadiroiban: Yes, I don't think the test suite uses x libs.  The javascript testing isn't integrated into test runs anyhow.20:11
adiroibanabentley: ok. thanks. Then I'll start configuring that machine.20:12
EdwinGrubbsabentley: I fixed the handling of the return value from find() and I pasted the incremental diff into https://code.edge.launchpad.net/~edwin-grubbs/launchpad/bug-433809-picker-search-slash/+merge/1567420:19
abentleyEdwinGrubbs: Cool.  On the phone.20:21
=== sinzui changed the topic of #launchpad-reviews to: on-call: abentley || reviewing: adiroban || queue [sinzui] || This channel is logged: http://irclogs.ubuntu.com || https://code.edge.launchpad.net/launchpad/+activereviews
thumperrockstar: shall I send a few reviews your way?20:33
rockstarthumper, sure.  Are they urgent?20:33
thumperthumper: isn't everything?20:34
* thumper facepalms20:34
thumperrockstar: I'll fire the ui one your way20:34
thumperrockstar: I just have others that you may be interested in20:35
=== matsubara is now known as matsubara-afk
rockstarthumper, okay, fire them my way.20:39
thumperrockstar: this is the ui one https://code.edge.launchpad.net/~thumper/launchpad/hiding-review-fields/+merge/1572220:40
thumperrockstar: that is just a "get the branch and look at it" type review20:40
thumperrockstar: and others around the claim/delete review arena:20:41
thumperhttps://code.edge.launchpad.net/~thumper/launchpad/claim-review-into-model-browser-integration/+merge/1571920:41
thumperhttps://code.edge.launchpad.net/~thumper/launchpad/claim-review-into-model-attempt2/+merge/1572820:41
thumperhttps://code.edge.launchpad.net/~thumper/launchpad/delete-pending-reviews/+merge/1573920:41
=== abentley 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
=== salgado is now known as salgado-afk
=== EdwinGrubbs is now known as Edwin-afk
bacsinzui: still around?23:20

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