On 4/10/2017 12:54 PM, Guido van Rossum wrote:
So the response from Martin Panter
(https://github.com/python/cpython/pull/851#issuecomment-292 )755992
sounds like he's not set up for the new GitHub workflow. I'm CC'ing
Martin here.
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.
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.
tjr
_______________________________________________
Core-mentorship mailing list
Core-mentorship@python.org
https://mail.python.org/mailman/listinfo/core-mentorship