<div dir="ltr">Thanks for the clarification. We should probably move this discussion to the python-committers list rather than core-mentorship.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 10, 2017 at 12:12 PM, Terry Reedy <span dir="ltr"><<a href="mailto:tjreedy@udel.edu" target="_blank">tjreedy@udel.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 4/10/2017 12:54 PM, Guido van Rossum wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
So the response from Martin Panter<br>
(<a href="https://github.com/python/cpython/pull/851#issuecomment-292755992" rel="noreferrer" target="_blank">https://github.com/python/cpy<wbr>thon/pull/851#issuecomment-292<wbr>755992</a>)<br>
sounds like he's not set up for the new GitHub workflow. I'm CC'ing<br>
Martin here.<br>
</blockquote>
<br></span>
The specific issue Martin raised is "Sorry but I don’t have an easy way to see your fixes relative to the old version I reviewed."  If the original and modified patches were posted in proper format to b.p.o., then one could hit [review] to start Rietveld and request a side-by-side diff of the two versions.  This is perfect for reviewing responses to comments, especially those made in-line.  For this issue, Martin made about 20 inline comments.<br>
<br>
I don't see any way to get the equivalent on a github PR.  It appears that the original patch is replaced by the revised patch.  To me, Rietveld was a great review tool and its loss a regression in the work process. I hope that this can be fixed somehow.<br>
<br>
tjr<div class="HOEnZb"><div class="h5"><br>
<br>
<br>
______________________________<wbr>_________________<br>
Core-mentorship mailing list<br>
<a href="mailto:Core-mentorship@python.org" target="_blank">Core-mentorship@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/core-mentorship" rel="noreferrer" target="_blank">https://mail.python.org/mailma<wbr>n/listinfo/core-mentorship</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">--Guido van Rossum (<a href="http://python.org/~guido" target="_blank">python.org/~guido</a>)</div>
</div>