[Libreoffice-qa] Gerrit auto-merge
Michael Meeks
michael.meeks at suse.com
Mon Jun 4 09:23:47 PDT 2012
On Sat, 2012-06-02 at 08:54 +0200, Lionel Elie Mamane wrote:
> Ah, that's very different. So essentially people that now have
> immediate gratification^W commit rights would "go back" to "one day
> delay"?
Nah - as Bjoern says this is only for non-trivial patches that the
reviewer is not confident about and that would prolly benefit from a day
of delay to allow a specialist in the given area to jump in and do the
work; clearly interacting / forwarding to the right person etc. is a
good idea here.
> something to the effect: "one day waiting period is a lot for eager
> new contributors". I think I thought this was about requests to apply
> a commit *already* in master to stable branch.
Heh ;-) it's about the above subset of patches that some are worried
about going into master without enough review.
Lets see how the gerrit flow helps as/when it comes. What I hated wrt.
the CWS process was it's huge amount of round-trips. If there is a
sausage machine that (as a committer) I can choose to push to, that
after building on <n> platforms auto-pushes that to master, I'd be dead
pleased. It means I can do more risky cross-platform changes more
quickly, and queue them up back to back - knowing I'm not going to bust
other people's builds ;-)
Of course, if we start finding new contributors' patches taking much
longer to merge, we should re-visit ;-) IIRC we decided to re-discuss
this at a later date.
HTH,
Michael.
--
michael.meeks at suse.com <><, Pseudo Engineer, itinerant idiot
More information about the Libreoffice-qa
mailing list