<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 8, 2015 at 9:25 AM, James Douglas <span dir="ltr"><<a href="mailto:jdouglas@wikimedia.org" target="_blank">jdouglas@wikimedia.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><span class="">> "A team should review their open patchsets before writing new code."<div><br></div></span><div>+1, this is a great way to remind ourselves to focus on priorities.<br></div></div></blockquote><div><br></div><div>While in principle I agree with this notion, I don't think this solves the problem of "<span style="font-size:12.8000001907349px">Our code review queues keep growing", or gets us closer to resolving the underlying issues. I would even go as far as to say that for many of the WMF engineering teams, staying focussed on their teams' priorities may in fact contribute to code review queues growing - especially for repositories that have no clear ownership or are for projects that do not rank near the top of the WMF's or a given team's priorities.</span></div><div><br></div><div>It's cool to see a "ranking of repositories" in Korma which ranks repos by "<span style="color:rgb(51,51,51);font-family:'Helvetica Neue',Helvetica,Arial,sans-serif;font-size:14px;line-height:19.999979019165px">oldest median age of unreviewed changesets". </span>I clicked through the first 100 of those repos and with only a few exceptions that I'm aware of, those repositories do not have owners on WMF engineering teams. Is there a way to see which repositories have the largest and/or fastest growing unreviewed queue of changesets? I would guess that the biggest issues around lack of code review happen in repos that have no clear/current ownership or are in repos owned by someone in the WMF but in a project that may be low organizational priority relative to other things they/their team is working on. But it's just a guess - seeing that kind of data might help us get a better understanding of the problem.</div><div><br></div><div>-- </div></div><div class="gmail_signature"><div dir="ltr">Arthur Richards<div>Team Practices Manager</div><div>[[User:Awjrichards]]</div><div>IRC: awjr</div><div>+1-415-839-6885 x6687</div></div></div>
</div></div>