<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Sep 19, 2015 at 12:07 PM, Boudhayan Gupta <span dir="ltr"><<a href="mailto:bgupta@kde.org" target="_blank">bgupta@kde.org</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 19 September 2015 at 21:17, Martin Graesslin <<a href="mailto:mgraesslin@kde.org">mgraesslin@kde.org</a>> wrote:<br>
> On Saturday, September 19, 2015 5:32:33 PM CEST Kevin Krammer wrote:<br>
>> If the problem is somewhere in (a), where is it?<br>
><br>
> I'm afraid of code review happening through the pull request instead of our<br>
> infrastructure. To me github pull requests are not just the "here's the<br>
> patch", but also the code review.<br>
<br>
</span>This.<br>
<br>
The other problem is that the PR submitter may not have a KDE<br>
identity, in which case we have no way of representing the fellow and<br>
properly crediting the commit to him/her. We have to explicitly<br>
redirect him to KDE's infrastructure for this.<br></blockquote><div><br></div><div>That is not true. You can credit any commit to anyone</div><div><br></div><div>git commit --author "My Name <<a href="mailto:myemail@hello.org">myemail@hello.org</a>>"</div><div><br></div><div>which is a standard practice around KDE, when committing patches</div><div>on behalf of newcomers who don't have write access.</div></div><div><br></div><div>Cheers</div>-- <br><div class="gmail_signature"><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div></div>
</div></div>