On Wed, Jun 1, 2011 at 5:28 PM, Chad <innocentkiller(a)gmail.com> wrote:
I don't think "revert in 72 hours if its
unreviewed" is a good idea. It
just discourages people from contributing to areas in which we only
have one reviewer looking at code.
I also don't really like it. But I do think we should be more liberal
about reverting things that won't be reviewed soon because they're
e.g. too large and should be branched or broken up, or whatever.
I *do* think we should enforce a 48hr "revert if
broken" rule. If you
can't be bothered to clean up your breakages in within 48 hours of
putting your original patch in, it must not have been very important.
+1. Serious breakage should be reverted on sight, however.
Roan Kattouw (Catrope)