<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Mar 14, 2016 at 12:40 PM, James Forrester <span dir="ltr"><<a href="mailto:jforrester@wikimedia.org" target="_blank">jforrester@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=""><div style="font-family:arial,helvetica,sans-serif"><span style="font-family:arial,sans-serif">On 14 March 2016 at 19:15, Greg Grossmeier </span><span dir="ltr" style="font-family:arial,sans-serif"><<a href="mailto:greg@wikimedia.org" target="_blank">greg@wikimedia.org</a>></span><span style="font-family:arial,sans-serif"> wrote:</span><br></div></span><div class="gmail_extra"><div class="gmail_quote"><span class=""><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">Backgroun:<br>
* This started as this task in our Phab:<br>
  <a href="https://phabricator.wikimedia.org/T121751" rel="noreferrer" target="_blank">https://phabricator.wikimedia.org/T121751</a> "Document best practices to<br>
  amend a change written by another contributor"<br>
* Lot's of discussion there about what is "right" in the general sense.<br>
* Mukunda found out a way of making everyone happy (maybe)<br>
* Mukunda proposed that solution upstream, then Evan P wrote a long<br>
  opinion piece on code review social contracts and basically concluded<br>
  that our social contracts are toxic (a theme we keep hearing...)<br>
** here: <a href="https://secure.phabricator.com/T10584" rel="noreferrer" target="_blank">https://secure.phabricator.com/T10584</a><br>
<br></blockquote></span><div><div><font face="arial, helvetica, sans-serif">[​I  pretty strongly disagree with Evan] </font><span style="font-family:arial,helvetica,sans-serif">Indeed, I'd be more blunt: I think it's actively disrespectful to peers – be they WMF staff, third party staff, or volunteers alike – to </span><b style="font-family:arial,helvetica,sans-serif">not</b><span style="font-family:arial,helvetica,sans-serif"> just tweak and merge their code. For instance, when there's a typo/misplaced whitespace in a comment or it needs a quick manual rebase, refusing to fix and merge isn't a great practice. People's time is precious and as reviewers we should all take on the burden of minor changes, and not give trivial C-1s (or Differential equivalents) that push the review cycle out for another hour/week/year (depending on the respondent's availability).</span></div></div></div></div></div></blockquote><div><br></div><div>I fully agree with this.  It troubles me to openly and strongly disagree with Evan, because I have so much respect for him as an upstream BDFL.  We have a lot to learn from him when it comes to being a healthy upstream.</div><div><br></div><div>That said, there is a very unhealthy contrarian attitude toward the competition (GitHub) which he is fostering.  If you look at the comment thread on T10584, you'll see some snark about GitHub's model being "broken".  Somehow, forcing developers to extra middleware like arc <i>isn't</i> broken, but GitHub is broken.  Riiiight.</div><div><br></div><div>In reading Evan's comments (and not other Phab users), his instincts seem to come from the Facebook new developer mentoring model, and may not be entirely wrong.  We need to evaluate whether we want to join him in this battle against the prevailing attitude popularized by GitHub.  Personally, I think it's a quixotic losing argument, but ultimately, WMF Release Engineering needs to decide where they are going to place their loyalty and support.</div><div><br></div><div>Rob</div><div><br></div></div></div></div>