[development] Intelligent use of time (Re: Perm for setting RTBC
Jeff Eaton
jeff at viapositiva.net
Tue Jul 3 15:53:29 UTC 2007
On Jul 3, 2007, at 9:56 AM, Fernando Silva wrote:
> 8. core developers will start to do their job of commiting patches
This is a point of much contention, however. Your statement implies
that a core committer is "The guy who's supposed to press the
'commit' button after everyone decides what should happen." With that
line of thinking, yes, a core committer is 'not their job' if they
don't commit RTBC patches.
The fundamental problem, though, is that a core committer is also
'the final reviewer,' the last line of defense between Drupal core
and conceptually flawed code. The system works best when core
committers don't *have* to spend a lot of time on that part of their
role. And that's what breaks when the queue gets saturated with
premature RTBC's. Their role as 'final reviewer' sucks up more and
more time, and fewer patches can get the attention necessary to make
the commit happen.
More information about the development
mailing list