Code review required for commits - Discuss.

Jelmer Vernooij jelmer at samba.org
Fri Oct 12 08:01:34 MDT 2012


On Fri, 2012-10-12 at 10:45 +0200, Volker Lendecke wrote:
> On Fri, Oct 12, 2012 at 01:41:01AM -0700, Matthieu Patou wrote:
> > On 10/12/2012 01:20 AM, Volker Lendecke wrote:
> > >On Fri, Oct 12, 2012 at 10:16:21AM +0200, Kai Blin wrote:
> > >>-----BEGIN PGP SIGNED MESSAGE-----
> > >>Hash: SHA1
> > >>
> > >>On 2012-10-12 09:38, Michael Adam wrote:
> > >>
> > >>Hi,
> > >>
> > >>>Ok, the main point is this:
> > >>>
> > >>>Change "review" from a matter of interest to a required step in
> > >>>our software development.
> > >>>
> > >>>Hence, I am inclinend to change my proposal to:
> > >>>
> > >>>* we should _not_ allow a free push pass after any number of days
> > >>>at all.
> > >>Ok, as the one who works on an area that sees little interest at all
> > >>until things break, how do I make sure I get my code in?
> > >Just be a PITA for the reviewers. Send mails, call them,
> > >start to use other means like hop on a train to Goettingen
> > >with a LART tool :-)
> > Of course I do understand the joke but in every joke there is a part
> > of truth, so what about people in .au or .us ? Should I knock on the
> > door of Jeremy to get a review ?
> > 
> > I don't think it's being fair to the developer to ask him to nag
> > reviewers in order to get a review. I do think that within 3 days a
> > reviewer should pop up to say: I want to review this and then maybe
> > the reviewer gets a bit more time to do the review but I still do
> > think (also) that it should be limited so that we keep on the
> > dynamic and review never get stuck for weeks as it has been the case
> > in the past for some voluntary reviews.
> 
> You are free to find someone else for review. Probably we
> need some incentive for reviewers. Maybe an ohloh category
> for "number of acks"?
My experience in other projects is that people who do more reviews are
also more likely to get reviews from others, and there is plenty of
pressure on those who don't do many reviews. I'm not sure if we need
more incentives.

Cheers,

Jelmer



More information about the samba-technical mailing list