[kde-community] What is a GitHub pull request exactly?

[Martin Graesslin]

On Saturday, September 19, 2015 6:40:47 PM CEST Kevin Krammer wrote:
> On Saturday, 2015-09-19, 17:47:38, Martin Graesslin wrote:
> > On Saturday, September 19, 2015 5:32:33 PM CEST Kevin Krammer wrote:
> > > If the problem is somewhere in (a), where is it?
> > 
> > I'm afraid of code review happening through the pull request instead of
> > our
> > infrastructure. To me github pull requests are not just the "here's the
> > patch", but also the code review.
> 
> But wouldn't that be just additional code review?
> 
> Either on top of no code review or on before actual code review for projects
> that require it?

do you really think that it would be reviewed again on KDE code review after 
it has gone through github review?

How would that process look like? Maintainer playing proxy by passing back the 
requested changes to the github developer?

No reality would be that it slowly moves code review from KDE to github. And I 
think we have here quite some people in the discussion who would love to see 
that :-(

Cheers
Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-community/attachments/20150919/62c8c0d1/attachment.sig>