<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Sun, 29 Nov 2015 at 21:08 Guido van Rossum <<a href="mailto:guido@python.org">guido@python.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Nov 29, 2015 at 7:54 PM, Nick Coghlan <span dir="ltr"><<a href="mailto:ncoghlan@gmail.com" target="_blank">ncoghlan@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>On 30 November 2015 at 03:12, Brett Cannon <<a href="mailto:brett@python.org" target="_blank">brett@python.org</a>> wrote:<br>
> Thanks for the feedback. And the "do nothing" option is there, although it's<br>
> so disliked by so many people that the chances of us not changing our<br>
> workflow is pretty slim.<br>
<br>
</span>The interests of folks that prefer the terminal focused<br>
"commit-locally-and-push" workflow can still be taken into account in<br>
the evaluation though - while it appears likely either GitHub or<br>
GitLab will be adopted as the repository management service, whether<br>
or not the maintenance branches and the default branch are marked as<br>
protected so even core developers *have* to go through the web based<br>
merge process is a separate question.<br></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>What?! I've never worked with a GitHub-based project where you *had* to use the web-based merge process. Hopefully that's not really on the table. In fact I'm not a big fan of GitHub's web-based merge process at all -- I much prefer seeing a simple linear history in the master (and I don't like preserving  intermediate commits made during the PR review process).<br></div></div></div></div></blockquote><div><br></div><div>Donald addressed the protected branch bits, but the web-based PR merging will be discussed as a possible allowed workflow. It doesn't have to be settled now but just so you know my position, I like the web-based merging as it means I don't have to worry about being on a machine with a repo and SSH keys in order to do a merge (e.g., I could do a merge from my Chromebook while on vacation or at work on my lunch break without issue). I also don't mind the intermediate merges as it gives contributors proper attribution for their work (you can use I believe `git log --merges` to only see git merge logs which would be written by core devs).</div><div><br></div><div>-Brett</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
There are also tools like git-pulls (Ruby:<br>
<a href="https://github.com/schacon/git-pulls" rel="noreferrer" target="_blank">https://github.com/schacon/git-pulls</a>) and hub (Go:<br>
<a href="https://hub.github.com/" rel="noreferrer" target="_blank">https://hub.github.com/</a>) that let folks review and merge GitHub PRs<br>
from the terminal. (I had a quick look through some of the command<br>
line clients listed at <a href="https://about.gitlab.com/applications/" rel="noreferrer" target="_blank">https://about.gitlab.com/applications/</a>, but<br>
didn't see anything as workflow focused as git-pulls or hub, so "good<br>
support for terminal based usage" may count as a concrete technical<br>
differentiator here)<br></blockquote></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"></div><br> Review and merge process should be separable. After 10+ years of using web-based review tools I personally wouldn't dream of using a terminal-based *review* (as opposed to merge) process. Though of course if that's your preference you should be able to do it.</div></div><div dir="ltr"><div class="gmail_extra"><br clear="all"><br>-- <br><div>--Guido van Rossum (<a href="http://python.org/~guido" target="_blank">python.org/~guido</a>)</div>
</div></div></blockquote></div></div>