[dmd-internals] To consume pull requests for trivial bugs
braddr at puremagic.com
Mon Aug 20 13:01:40 PDT 2012
On Mon, 20 Aug 2012, Andrei Alexandrescu wrote:
> On 8/20/12 2:31 PM, David Nadlinger wrote:
> > On Mon, Aug 20, 2012 at 8:09 PM, Brad Roberts<braddr at puremagic.com> wrote:
> > > Sorry, but the fix to not enough people with front end experience doing
> > > code reviews is not to not do code reviews.
> > Agreed, but I'd like to add that this is a general problem with DMD
> > development (maintainability?) - even Walter regularly commits code
> > which doesn't even pass the auto tester.
> This issue is a symptom of a simple underlying problem: we don't have enough
> contributors. Kenji tried to find a solution within the current constraints,
> whereas Brad responded from an "enough reviewers" frame of reference.
> I think we need to add more active committers to dmd.
I wasn't suggesting anything other than 'less code revew' isn't a solution
that's in our best interest.
If you want my opinion, Walter shouldn't be writing code unless the review
queue is empty. It's harsh, but it'd net us far better throughput of
fixes and keep the contributors contributing. We loose contributors and
constrict their contribution rate by not allowing work to flow into the
With increased throughput and increased contributors, they'd be in a
better position to also review either others code, making the need for
Walter to be the reviewer lower.
Lastly, as part of the code review, Walter needs to stop running the
tests. That's what the pull auto tester is for. Look at the code,
comment where not ready, merge where ready.
More information about the dmd-internals