<div dir="ltr">> "A team should review their open patchsets before writing new code."<div><br></div><div>+1, this is a great way to remind ourselves to focus on priorities.<br><br>This is also basic product management; if I'm working on B while the patch for A sits around unused, I'm implying that B is a higher priority than A.  That may be true -- priorities do change from time to time, but maybe not.  Either way, it's a loud signal that I might need to contact my PM for clarification, or my Scrum master to help clear the blocker on A.<br><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 8, 2015 at 3:42 AM, Quim Gil <span dir="ltr"><<a href="mailto:qgil@wikimedia.org" target="_blank">qgil@wikimedia.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Mon, Jun 8, 2015 at 12:35 PM, Antoine Musso <span dir="ltr"><<a href="mailto:hashar+wmf@free.fr" target="_blank">hashar+wmf@free.fr</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">When I look at the korma page, the two first pages are for MediaWiki<br>
extensions that have been abandoned ages ago.  Either because it was a<br>
one time proof of concept or it has migrated to GitHub.   If we could<br>
archive / move to an attic all the bitrotting repositories that will<br>
surely help.<br></blockquote><div><br></div></span><div>If that is true, then the old open reviews that are making those repositories show up can be safely abandoned, right? There is no point in keeping open reviews in abandoned / migrated repositories.</div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
mediawiki/core itself has 500+ open changes or a good third of the debt.<br>
  If one could generate a list of changes per authors and then ask each<br>
author to move its patches forward, that would help.<br>
<br>
Remember the main reason to have a premerge review workflow was to put<br>
the responsibility of review in the hands of the author.  If people<br>
don't babysit their patches, there is not much we can do I guess.<br></blockquote><div><br></div></span><div>As said, -1 and WIP are already filtered in those metrics. The patches you see are either 0 or +1. The way to tell authors to move patches forward is by posting -1 with a comment of what is missing.</div><div><br></div></div><span class="">-- <br><div>Quim Gil<br>Engineering Community Manager @ Wikimedia Foundation<br><a href="http://www.mediawiki.org/wiki/User:Qgil" target="_blank">http://www.mediawiki.org/wiki/User:Qgil</a></div>
</span></div></div>
<br>_______________________________________________<br>
teampractices mailing list<br>
<a href="mailto:teampractices@lists.wikimedia.org">teampractices@lists.wikimedia.org</a><br>
<a href="https://lists.wikimedia.org/mailman/listinfo/teampractices" target="_blank">https://lists.wikimedia.org/mailman/listinfo/teampractices</a><br>
<br></blockquote></div><br></div>