On Wed, Jun 1, 2011 at 5:28 PM, Chad innocentkiller@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)