<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Jul 10, 2018 at 2:00 AM Terry Reedy <<a href="mailto:tjreedy@udel.edu">tjreedy@udel.edu</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 7/9/2018 9:00 PM, Guido van Rossum wrote:<br>
> We strongly prefer feedback in the form of Pull Requests to the peps <br>
> repo (the file is at <br>
> <a href="https://github.com/python/peps/blob/master/pep-0572.rst" rel="noreferrer" target="_blank">https://github.com/python/peps/blob/master/pep-0572.rst</a> <br>
> <<a href="https://github.com/python/peps/blob/master/pep-0572.rst" rel="noreferrer" target="_blank">https://github.com/python/peps/blob/master/pep-0572.rst</a>>).<br>
<br>
I couple of people have said they don't know how to make pull requests <br>
against a PEP.  I believe the following works with the web editor.<br>
<br>
1. Go to url above.<br>
2. Click upper right pencil icon.  I don't know just who can do this.<br>
3. Edit on the web page.  It might be easier to read the html and then <br>
use browser find to change anything seen.<br>
4. Note that there are two scrollbars -- one for webpage and one for <br>
editing box.<br>
5. At the bottom of the webpage, click 'Create a new branch for this <br>
commit and start a PR'.  This should unclick 'Commit directly to the <br>
master branch.'<br>
6. Click Green [Propose file change] button.<br>
7. I expect one will end up on the PR diff page.  If not, click 'files <br>
changed to get there.  Add comments explaining changes that need such.<br></blockquote><div><br></div><div>This is almost precisely how I, a newbie to contributing, fixed a typo in this PEP tonight. At step 2, anyone who is logged in can click the pencil icon; if you don't have write access to the repo, GitHub automatically forks the repo and commits your change to a new branch within your fork, eliminating step 5.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The problem I see with this is that I don't know of any way to accept or <br>
reject the different proposed changes within the PR.  If there is, I <br>
would like to know how.<br></blockquote><div><br></div><div>I don't believe there is a way to merge part of a PR. The rejected changes would have to be reverted with a new commit prior to merging.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
-- <br>
Terry Jan Reedy<br>
<br>
_______________________________________________<br>
Python-Dev mailing list<br>
<a href="mailto:Python-Dev@python.org" target="_blank">Python-Dev@python.org</a><br>
<a href="https://mail.python.org/mailman/listinfo/python-dev" rel="noreferrer" target="_blank">https://mail.python.org/mailman/listinfo/python-dev</a><br>
Unsubscribe: <a href="https://mail.python.org/mailman/options/python-dev/jcgoble3%40gmail.com" rel="noreferrer" target="_blank">https://mail.python.org/mailman/options/python-dev/jcgoble3%40gmail.com</a><br>
</blockquote></div></div>