[dmd-internals] To consume pull requests for trivial bugs
braddr at puremagic.com
Mon Aug 20 11:09:36 PDT 2012
I strongly disagree with your implication that the auto-tester counts as
any sort of a review. It can only detect incorrectness, not declare
correctness. There's little evidence to suggest that anyone is going to
look at the code post-commit and I'm not comfortable with hoping that
someone will. Hoping that someone's testing happens to catch them during
a very tiny beta period (a different problem unto itself) is just fantasy.
Sorry, but the fix to not enough people with front end experience doing
code reviews is not to not do code reviews.
My 2 cents,
On Mon, 20 Aug 2012, kenji hara wrote:
> I had been invited as dmd committer a month ago, but haven't been able
> to consume pull requests.
> The top three of pulls at 2012/08/20:
> 37 pulls : my front-end changes
> 17 pulls : yebblies's front-end changes
> 13 pulls : dawgfoto's backend and front-end changes
> -> 67/103 (65% of all pulls)
> and, as far as I seen half of them are trivial bug fixes and LGTM.
> But they are yet not merged by the following reasons.
> #1. Some of them are mine, so I cannot merge without others reviewing.
> #2. Some of them are already outdated, so I cannot merge them without
> small fix-up.
> But, even if I add some fix-ups to them, I can not merge them by
> the reason #1 .
> I think the most problem is too few reviewers for dmd, but it's
> difficult to solve in short term.
> To solve the deadlocks, I'd like to suggest development process improvements.
> For #1:
> If the pull request passes the auto tester, I can merge without others
> This is based on "reviewing after committing", but it seems to me that
> would not be a problem,
> because current dmd release cycle has "beta-phase" to fix regressions.
> For #2:
> I would add fixups in my local, and post pull requests which contains
> it, then treats them as #1.
> This is for only _trivial_ bug fixes, and not for big ones and enhancements.
> How about?
> dmd-internals mailing list
> dmd-internals at puremagic.com
More information about the dmd-internals