<div dir="ltr"><p style="margin:0px 0px 12px;padding:0px;border:0px;color:rgb(0,0,0);font-family:'Segoe UI','Segoe UI Web Regular','Segoe UI Symbol','Helvetica Neue',Helvetica,Arial,sans-serif;font-size:13px;line-height:18.8500003814697px">Thank you for this discussion! I have updated the description of <a href="https://phabricator.wikimedia.org/T101686">https://phabricator.wikimedia.org/T101686</a> based on the feedback received, and will continue to do so.</p><p style="margin:0px 0px 12px;padding:0px;border:0px;color:rgb(0,0,0);font-family:'Segoe UI','Segoe UI Web Regular','Segoe UI Symbol','Helvetica Neue',Helvetica,Arial,sans-serif;font-size:13px;line-height:18.8500003814697px">I'm thinking of morphing this proposal into a WMF committed goal: <strong>The code review queue must stop growing and start decreasing</strong></p><p style="margin:0px 0px 12px;padding:0px;border:0px;color:rgb(0,0,0);font-family:'Segoe UI','Segoe UI Web Regular','Segoe UI Symbol','Helvetica Neue',Helvetica,Arial,sans-serif;font-size:13px;line-height:18.8500003814697px">Then each team and each individual would be free to take the approaches that fit best with their workflows. Meanwhile, Team Practices and Community Engineering could work on recommendations, activities, and metrics supporting that goal, and product managers and line should be onboard when planning new goals and sprints.</p><p style="margin:0px 0px 12px;padding:0px;border:0px;color:rgb(0,0,0);font-family:'Segoe UI','Segoe UI Web Regular','Segoe UI Symbol','Helvetica Neue',Helvetica,Arial,sans-serif;font-size:13px;line-height:18.8500003814697px">So far we have identified several problems (listed in T101686). <span style="line-height:18.8500003814697px">As we dig into the problems, we will identify specific problems and hopefully their specific remedies. Does this sound better?</span></p><p style="margin:0px;padding:0px;border:0px;color:rgb(0,0,0);font-family:'Segoe UI','Segoe UI Web Regular','Segoe UI Symbol','Helvetica Neue',Helvetica,Arial,sans-serif;font-size:13px;line-height:18.8500003814697px">I have proposed in <a href="https://phabricator.wikimedia.org/T101099" class="" style="text-decoration:none;color:rgb(0,0,0);font-weight:bold"><span class="" style="color:inherit;border:1px solid rgb(231,231,231);border-radius:3px;padding:0px 4px;background-color:rgb(231,231,231)">T101099: Engineering Community Roadmap</span></a> a sequence of goals to solve the code review time wait problem in the next 12 months. <a href="https://phabricator.wikimedia.org/T88531" class="" style="text-decoration:none;color:rgb(0,0,0);font-weight:bold"><span class="" style="color:inherit;border:1px solid rgb(231,231,231);border-radius:3px;padding:0px 4px;background-color:rgb(231,231,231)">T88531: Goal: Organize a Gerrit Cleanup Day</span></a> would be the first step (awareness and official kick-off of the discussion), followed by changing the trend (queue stops growing), cleaning up, and committing to some kind of service-level agreement. I (or <a href="https://phabricator.wikimedia.org/p/Aklapper/" class="" style="text-decoration:none;color:rgb(25,85,141);font-weight:bold;white-space:nowrap"><span class="" style="color:inherit;border:1px solid rgb(241,247,255);border-radius:3px;padding:0px 4px;background-color:rgb(241,247,255)">@Aklapper</span></a>, or whoever is faster) will create specific tasks to discuss these goals as soon as we have enough feedback supporting this direction.</p></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 10, 2015 at 10:41 AM, Gilles Dubuc <span dir="ltr"><<a href="mailto:gilles@wikimedia.org" target="_blank">gilles@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"><div>I think this is a cultural problem, which means that it will be difficult to solve by forcibly pushing people towards a specific way of fixing it. It's not a staff vs volunteer nor an active project vs abandoned project issue. Other open source projects with worse staff to volunteers ratios don't have that problem. We have to change the conditions and the landscape to produce a long-lasting cultural shift.<br><br></div>The way I try to contribute at my own little scale is to always aim to review more changesets than I produce. If everybody did that, the issue would go away, right? And it doesn't have to be tied to a specific workflow, like reviewing before writing, etc. which wouldn't fit everybody and wouldn't even fit me every day. It doesn't matter how you organize your schedule as long as at the end of the week/month/year you've reviewed more changes than you've asked others to review.<br><br>Maybe that specific way of seeing things (am I in review debt?) could be supported by software. I.e. a prominent indicator of whether or not you've pushed more changesets than you've reviewed. It might be something to consider once we move code reviews to phabricator. Moving away from gerrit should be a priority, in fact, as the increased ease of use and integration of our review tool could have an impact on those figures.<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 10, 2015 at 2:00 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span><div> > Exactly. And thus a team prioritizing Bar might continue coding on Bar, 
allowing the queue of pending Foo code reviews to pile up.<br><br></div></span>Bingo!<br> </div><div class="gmail_extra"><br><div class="gmail_quote"><span>On Tue, Jun 9, 2015 at 1:55 PM, Kevin Smith <span dir="ltr"><<a href="mailto:ksmith@wikimedia.org" target="_blank">ksmith@wikimedia.org</a>></span> wrote:<br></span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><div dir="ltr"><span><br><div class="gmail_extra"><div class="gmail_quote">On Tue, Jun 9, 2015 at 1:10 PM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">This prioritization then makes it trivial to decide between reviewing (or fixing, testing, deploying, etc.) feature Foo vs. coding up feature Bar.<br></div><br></blockquote></div><br></div></span><div class="gmail_extra">Exactly. And thus a team prioritizing Bar might continue coding on Bar, allowing the queue of pending Foo code reviews to pile up. <br><span><font color="#888888"><br></font></span></div><span><font color="#888888"><div class="gmail_extra">Kevin<br><br></div></font></span></div>
<br></div></div><span>_______________________________________________<br>
teampractices mailing list<br>
<a href="mailto:teampractices@lists.wikimedia.org" target="_blank">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></span></blockquote></div><br></div>
<br>_______________________________________________<br>
teampractices mailing list<br>
<a href="mailto:teampractices@lists.wikimedia.org" target="_blank">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>
</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><br clear="all"><div><br></div>-- <br><div class="gmail_signature">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>
</div>