On Wed, Jun 1, 2011 at 5:28 PM, Chad <innocentkil...@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) _______________________________________________ Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l