Rockbox mail archive
Subject: RE: Git/gerrit migration status and next steps
RE: Git/gerrit migration status and next steps
On Sep 7, 2011 5:43 PM, <bryan.childs_at_rbs.com> wrote:
> > Do we really want to allow commits directly? so little of our code ever
> > reviewed so I would quite happily force everyone to go through gerrit
> > require someone else to OK it. It isnt hard to get someone else in IRC
> > a quick look and push the button.
> Absolutely against requiring this.
> We trust committers to do "the right thing".
> If you have a patch you think needs this - submit to refs/for/master.
> For small commits, this would be a completely pain in the arse, and
> Bryan Childs
This isn't about people doing the wrong thing. This is about trying to get
code quality. And yes it will be a minor pain sometimes, but a small fix has
just as much fuck-up-ability as a big one and is *easier* to sanity check
then the required debugging later.
This is something gerrit makes easy and we should absolutly do it. Remember
that you push directly to gerrit, this doesn't require you to open your
Received on 2011-09-07
Page was last modified "Jan 10 2012" The Rockbox Crew