=== Moot2 is now known as MootBot === allenap1 is now known as allenap === mrevell is now known as mrevell-lunch === mrevell-lunch is now known as mrevell === salgado-brb is now known as salgado [15:00] #startmeeting [15:00] Meeting started at 09:00. The chair is barry. [15:00] Commands Available: [TOPIC], [IDEA], [ACTION], [AGREED], [LINK], [VOTE] [15:00] hi everybody and welcome to this week's reviewers meeting. who's here today? [15:00] me [15:00] me [15:00] me [15:01] me [15:01] me [15:01] me [15:01] me [15:01] me [15:01] me [15:02] me [15:03] BjornT_ danilos ping [15:03] me [15:03] (I have my reminder set for an hour earlier, and then always miss out the real beginning; with DST soon to be off, I should be a regular :)) [15:04] mars: rockstar ping [15:04] me [15:05] [TOPIC] agenda [15:05] New Topic: agenda [15:05] * Roll call [15:05] * If there's time, the old boring script [15:05] * Next meeting [15:05] * f2f at epic? [15:05] * Action items [15:05] * Queue status [15:05] * Mentoring update [15:05] [TOPIC] next meeting [15:05] New Topic: next meeting [15:05] corner of Kings Cross and... [15:06] since we'll all be in london the next two weeks, i propose we suspend the irc meetings and try to find a time to meet f2f [15:06] +1 [15:06] danilos: are you planning on kerb crawling then? :) [15:06] we can talk about things to make the meetings better, improve pollination w/asiapac, ways to improve review process, etc [15:06] barry: +1 [15:07] +1 [15:07] +1 [15:07] +1 [15:07] bigjools: Either that or he'll be getting mugged, depending on whom he encounters first. [15:07] gmb: the pimp or the ho [15:07] ouch [15:07] :) [15:07] gmb: I am going for an attempted robbery, i.e. someone should try to rob me :) [15:07] bigjools: Mind you, danilos is a big fella, he should be safe. [15:07] Hah. [15:08] danilos: My point exactly ;) [15:08] [AGREED] suspend next two weeks irc in favor of f2f in london [15:08] AGREED received: suspend next two weeks irc in favor of f2f in london [15:08] [TOPIC] action items [15:08] New Topic: action items [15:08] * flacoste and foundations to look into techniques for eliminating back-patching of schema types (avoiding circular imports) [15:08] carry over [15:09] done [15:09] * barry will move the preimp discussion to the ml [15:09] i suck and propose we just any related issues over at epic [15:10] * flacoste to take discussion of rest v. moin to ml [15:10] flacoste: any word on gary's experiment? [15:10] when you push things to the epic, it's probably a good idea to try and actually schedule it. from what i saw there's a pretty dense plan [15:10] gary is progressing [15:10] he should be finished today [15:10] intellectronica: good oint [15:10] and we will send an email to the list then [15:10] flacoste: excellent, thanks [15:10] [ACTION] barry will schedule reveiwers meeting at epic [15:10] ACTION received: barry will schedule reveiwers meeting at epic [15:11] * rockstar to take discussion of adding launchpadlib tests for exposed api to ml [15:12] * barry pokes rockstar [15:12] fwiw i just got another bug in today which only happens with lplib and which tests like this would have caught ;) [15:13] oh well, let's move on then ;) [15:13] * abentley to investigate current code coverage tools for lp tests [15:13] I looked at the existing stuff, and it seems like the opposite of what I want. [15:14] I want to know if the tests run by a test command exercise all new code. [15:14] The existing work is about determining which tests exercise a given line of code. [15:14] abentley: at what granularity? [15:15] abentley: so, method or line of code for the opposite direction as well? [15:15] danilos: Ideally, just yes/no granularity. Either the changes are properly tested or they're not. [15:17] So the obvious implementation is: run the tests with coverage calculation, determine which lines are new, make sure they're covered. [15:17] abentley: tho iwbn to know which lines /aren't/ covered [15:18] barry: Sure. [15:18] abentley: any thoughts on putting something like this together? [15:19] barry: You mean, am I willing to do it? [15:19] abentley: jfdi, man! [15:21] barry: I guess I could hack on it on review day. [15:21] abentley: in what way does --coverage do the opposite of you want? it seems to tell you which lines are tested, and which aren't. that should be the first step in what you want, isn't it? [15:21] BjornT_: I'm not saying --coverage does. [15:22] abentley: oh. you said the existing tools did the opposite. i assumed you meant --coverage. [15:22] BjornT_: I was talking about launchpad-specific work that someone did that I was pointed at last week. [15:23] ah, right [15:23] abentley: well if you have time, i think it would be a useful tool [15:25] anyway, that's all i have for today. does anybody have topics not on the agenda? [15:25] BjornT_: https://launchpad.canonical.com/TestsFromChanges [15:27] okay then, thanks everyone and see you next time in london! [15:27] #endmeeting [15:27] Meeting finished at 09:27. [15:28] see y'all in The Smoke === salgado is now known as salgado-lunch === cprov is now known as cprov-lunch === salgado-lunch is now known as salgado === cprov-lunch is now known as cprov === salgado is now known as salgado-afk [23:36] hi [23:36] barry isn't here :)