[CRIU] github pull-requests

[Mike Rapoport]

On Fri, Feb 01, 2019 at 09:45:36AM -0800, Andrei Vagin wrote:
> Hi All,
> 
> 
> Let's start this experiment. Starting from today, you can create
> pull-requests on github. And it is a preferred way to submit changes.
 
And how github pull requests are supposed to handle Acked-by and
Reviewed-by? :-O

> On Tue, Nov 20, 2018 at 12:22:21AM -0800, Andrei Vagin wrote:
> > Hello,
> > 
> > On the Linux Plumbers Conference, I heard a few requests to start
> > accepting github pull-requests. Then we discussed this on the first
> > official criu hackathon and it looks like there were no objections. I
> > know that many of you were not able to be there, so I decided to start
> > this thread where you can share thoughts about the subject.
> > 
> > Pros:
> > * It is easier to create a pull-requests than send a patch. Stop,
> > stop, stop. Don't laugh. You did this many times, but for a new
> > contributor, it is a real problem.
> > * We can remove all our machinery, what is used validate patches.
> > 
> > Cons:
> > * The standard workflow is changed for people who read patches in the
> > mailing list.
> > * No multi-thread discussions
> > 
> > https://begriffs.com/posts/2018-06-05-mailing-list-vs-github.html
> > 
> > Questions:
> > 1. Should we start accepting github pull requests?
> > 2. Should we stop accepting patches from a mailing list?
> > 3. Can we do both?
> > 4. Which path should be a preferred one?
> > 5. Do we need a robot which will send github pull-requests into the
> > mailing list?
> > https://github.com/google/pull-request-mailer
> > 6. Do we need a robot which will create github pull-requests from patch series?
> > 7. Should we sync comments between the mailing list and github?
> > 8. Who wants to implement 5., 6. and 7.?
> > 
> > My answers:
> > 1. yes
> > 2. no
> > 3. yes
> > 4. the mailing list
> > 5. yes
> > 6. would be good
> > 7. would be good
> > 8. I don't :)
> > 
> > Thanks,
> > Andrei Vagin
> _______________________________________________
> CRIU mailing list
> CRIU at openvz.org
> https://lists.openvz.org/mailman/listinfo/criu
> 

-- 
Sincerely yours,
Mike.