<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 3, 2017 at 10:56 AM, Shyam <span dir="ltr"><<a href="mailto:srangana@redhat.com" target="_blank">srangana@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 03/01/2017 05:22 PM, Niels de Vos wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Wed, Mar 01, 2017 at 02:43:44PM -0500, Shyam wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
We do not accept github PRs as our work flow is gerrit based.<br>
<br>
To educate contributors, one of the things we could do is to put up a<br>
template as in [1] for the PULL_REQUEST_TEMPLATE. It will look like this [2]<br>
when users attempt to submit PRs.<br>
<br>
An alternate idea suggested by Nigel was to accept PRs, but use a Jenkins<br>
job to auto close the same, with some generic (or very specific)<br>
instructions on how to submit it to our gerrit instance.<br>
<br>
The advantage of the latter method is that we do not lose the code change<br>
suggested, and we can still evaluate the same and reach out to the<br>
contributor and also possibly get the change into our gerrit instance.<br>
<br>
The latter approach looks good, wanted to run this by other maintainers for<br>
their thoughts.<br>
</blockquote>
<br>
How many PR's do we get? If it is 'a lot', then the time to invest in<br>
the 2nd approach would be ok, otherwise a simple template is probably<br>
sufficient (and Nigel can spend his time on more urgent tasks).<br>
</blockquote>
<br></span>
We do not get many PRs.<br>
<br>
There is no urgency hence to do this, but doing the latter, give us the advantage of not losing the proposed code, and will be nicer to contributors coming in via github.<br>
<br>
I will let Nigel comment about his time, as I have clarified (my) priorities here.<br>
<br>
What about the thought itself, does it merit that we post it as a request to the infra team? I am not hearing otherwise, and I sure think it is useful.<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div><br></div><div><br></div><div>+1. I think the idea is useful.</div><div><br></div><div>Cheers,</div><div>Vijay </div></div></div></div>