<div dir="ltr">[It seems to me most of this thread has gone off-topic since it now includes what I consider a sexist comment, complaining about git, and discussing people's credentials in terms of being able to comment on something, I'm basically ignoring everything that came after this email as none of that has anything to do with what Antoine is complaining about.]<br><div><div><br><div class="gmail_quote"><div dir="ltr">On Sun, 25 Jun 2017 at 01:34 Antoine Pitrou <<a href="mailto:solipsis@pitrou.net">solipsis@pitrou.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sat, 24 Jun 2017 21:37:46 -0700<br>
Larry Hastings <<a href="mailto:larry@hastings.org" target="_blank">larry@hastings.org</a>> wrote:<br>
> On 06/24/2017 09:14 PM, Serhiy Storchaka wrote:<br>
> > Not only core developers make PRs for CPython. Since all non-trivial<br>
> > changes need to be mentioned in Misc/NEWS, blurb becomes a required<br>
> > tool for all committers.<br>
><br>
> Well, no.  *Writing blurb-compatible files* becomes a required step for<br>
> all committers.  And blurb makes that easy.  But it's pretty easy to<br>
> write them by hand; that's why we pre-created the "next" directories,<br>
> and there are instructions in the dev guide.<br>
<br>
Hmm.  If it were so easy, you wouldn't have felt the need to add that<br>
functionality to blurb, right? :-)<br></blockquote><div><br></div><div>Actually a good amount of blurb is to handle blowing up a Misc/NEWS file into individual news files and then re-combining them back into a Misc/NEWS file for release. The format also simplified over time while we refined the design so that the file name contains the most common metadata, thus not really needing to check the formatting as much.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
This is touching a more general problem, though.  Before GitHub, we<br>
(core developers) would take the patch submitted by a contributor, make<br>
whatever minor changes were needed (e.g. Misc/NEWS) and push the<br>
aggregate ourselves.  With GitHub, while it's possible to edit someone<br>
else's PR, it's frankly a PITA (I've tried to do it once, I don't want<br>
to try a second time unless GitHub makes it massively easier and less<br>
footgunning-prone).</blockquote><div><br></div><div>So there's editing a PR by adding/removing files and then there's editing files that have already been touched by the PR. In the latter case you can just edit the files through a browser while reviewing them no problem (I do this regularly rather than ask a PR submitter to fix e.g. a spelling mistake, just click the pencil icon at the top-right corner of the diff for the file).<br><br>In the former case where you want to add/delete a file is where it's less obvious on how to do it. Donald has said how to pull down a pull request as a branch in your clone, editing, and then pushing again. But it turns out you can <i>also</i> do it entirely through the web UI. For instance, if you look at your <a href="https://github.com/pitrou/cpython/tree/signal_pyatomic">https://github.com/pitrou/cpython/tree/signal_pyatomic</a> branch you had for your <a href="https://github.com/python/cpython/pull/2417">https://github.com/python/cpython/pull/2417</a> PR (sorry about that, but I needed someone's branch to test with that wasn't my own), you will see I created and deleted a news file, and I did that entirely through my browser. The biggest pain, though, is navigating back to someone's branch as I don't think there's a direct link to be found in a PR back to the repo:branch it's based on.<br><br></div><div>IOW everything you used to do by hand can still be done by hand, just through a web browser ... on a tablet at the beach with WiFi (which was one of my goals of this whole transition to begin with :) .<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> So we have to rely on contributors to make the PR<br>
merge-ready by themselves... which means spending some time guiding<br>
them through the oh-so-exciting steps necessary to add the right<br>
Misc/NEWS entry,</blockquote><div><br></div><div>My hope is that having a news entry status check through Bedevere with help with that. I'm also open to ideas on how to make it more obvious to first-time contributors that things need to be done, e.g. maybe a default PR message that is nothing more than "<!-- FIRST-TIME CONTRIBUTORS: make sure to visit the CONTRIBUTING link above as CPython deviates from typical GitHub practices. -->" (making it an HTML comment means it won't' ever show up in the rendered output of the PR, plus that comment is short and thus easy to ignore)? We could also make the CONTRIBUTING.md file have a bullet-point list of the key things we expect people to do that is different from most projects. (The only other thing I can think of is a comment for first-time contributors pointing all of this out, but I'm leery of that as all of our bots are stateless and this would make at least Bedevere stateful.)<br><br></div><div>And if you want to make it easier to add/remove files in a PR, we could look at adding a link to the branch the PR comes from so that it's easy to get to to do things like add a news file through the browser.<br></div><div><br></div><div>If any of this sounds reasonable we can discuss it over on core-workflow.<br></div><div><br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> or fix the occasional bit of reStructuredText<br>
mis-syntax.<br>
<br></blockquote><div><br></div><div>You can make those sorts of changes in the browser yourself.<br><br></div><div>-Brett<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Regards<br>
<br>
Antoine.<br>
<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/brett%40python.org" rel="noreferrer" target="_blank">https://mail.python.org/mailman/options/python-dev/brett%40python.org</a><br>
</blockquote></div></div></div></div>