<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, 15 Mar 2017 at 08:44 Berker Peksağ <<a href="mailto:berker.peksag@gmail.com">berker.peksag@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Mar 13, 2017 at 12:11 AM, Brett Cannon <<a href="mailto:brett@python.org" class="gmail_msg" target="_blank">brett@python.org</a>> wrote:<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> On Sun, 12 Mar 2017 at 06:33 M.-A. Lemburg <<a href="mailto:mal@egenix.com" class="gmail_msg" target="_blank">mal@egenix.com</a>> wrote:<br class="gmail_msg">
>><br class="gmail_msg">
>> On 10.03.2017 23:13, Brett Cannon wrote:<br class="gmail_msg">
>> > Fifth, anything I missed? :)<br class="gmail_msg">
>><br class="gmail_msg">
>> My main nit after the move is that messages to the checkin list<br class="gmail_msg">
>> no longer include the full patch. This makes reviews harder than<br class="gmail_msg">
>> necessary (you always have to go through the browser).<br class="gmail_msg">
>><br class="gmail_msg">
>> Is there some way this could be changed back to what we had<br class="gmail_msg">
>> previously or is this a hard limitation of github ?<br class="gmail_msg">
><br class="gmail_msg">
><br class="gmail_msg">
> It's a hard limitation of the GitHub-provided email solution. With GitHub's<br class="gmail_msg">
> APIs and enough time someone could either come up with a custom email<br class="gmail_msg">
> solution or a web page that showed this information (you literally just need<br class="gmail_msg">
> to add ".diff" to the end of a URL to get the diff itself for a PR, e.g.<br class="gmail_msg">
> <a href="https://github.com/python/cpython/pull/648.diff" rel="noreferrer" class="gmail_msg" target="_blank">https://github.com/python/cpython/pull/648.diff</a> will redirect to a raw<br class="gmail_msg">
> diff). There might also already be other solutions out there that do what<br class="gmail_msg">
> you're after.<br class="gmail_msg">
><br class="gmail_msg">
> Obviously this all requires work on someone's part. :) (I've also moved<br class="gmail_msg">
> completely off of an email-based workflow so I'm definitely not the right<br class="gmail_msg">
> person to drive this sort of thing.)<br class="gmail_msg">
<br class="gmail_msg">
I wrote <a href="https://github.com/berkerpeksag/cpython-emailer-webhook" rel="noreferrer" class="gmail_msg" target="_blank">https://github.com/berkerpeksag/cpython-emailer-webhook</a> to<br class="gmail_msg">
solve this problem. It works and its output is almost same as the old<br class="gmail_msg">
one [1] I even wrote some tests and documentation, but I just noticed<br class="gmail_msg">
that I forgot to push to GitHub :)<br class="gmail_msg">
<br class="gmail_msg">
If we all agree on the idea I can help with deploying it. I can use my<br class="gmail_msg">
own VPS for the initial deploy, but it would be nice to have a PSF<br class="gmail_msg">
backed server in the long term.<br class="gmail_msg"></blockquote><div><br></div><div>I believe we have free Heroku resources to putting it there should be easy (the infrastructure team would know best obviously where we have free resources).</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">
<br class="gmail_msg">
--Berker<br class="gmail_msg">
<br class="gmail_msg">
[1] <a href="https://github.com/python/core-workflow/issues/24#issuecomment-279162079" rel="noreferrer" class="gmail_msg" target="_blank">https://github.com/python/core-workflow/issues/24#issuecomment-279162079</a><br class="gmail_msg">
</blockquote></div></div>