Re: Gets rid of length in the docs. (issue 4965053)

[Trevor Daniels]

Graham Percival wrote Monday, August 29, 2011 9:55 AM


> On Mon, Aug 29, 2011 at 09:15:54AM +0100, Trevor Daniels wrote:
> > If you force patches through at the present rate
> > they're not going to be carefully reviewed.
>
> I'm open to suggestions.  The development team is producing
> approximately 20 patches (including drafts) per week.  We could
> always tell some programmers to bugger off, but that seems highly
> counter-productive.  :)

Indeed.  Perhaps we could make a clearer distinction
between patches that must be reviewed, like those
which cause syntax changes, those which add new
features and those by relative newcomers, and those
that should simply be pushed without review, like
bug fixes by experienced developers, minor doc changes,
etc, with the aim of reducing further the number going 
to Reitveld and lengthening the queue.  Another GOP 
topic?

> Another idea would be to try forcing people to review patches.  I
> don't see that going well, either.

No, sounds pretty impractical too.

Trevor