On 01.06.2011, 20:00 Roan wrote:
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.
+1. "Serious" should include "breaking the tests" though -
otherwise
we'll remain with our current situation when tests are broken
in multiple places and nobody knows who broke what:
[19:39:08] <codurr> Something broke. See
<http://ci.tesla.usability.wikimedia.org/cruisecontrol/buildresults/mw>. Possible
culprits: aashrh/r89027 /r89028 /r89029 nbiabriket/r89035 krenkli/r89036 eryed/r89037
/r89038 eixal/r89039 /r89040 /r89041 /r89043 /r89044 /r89047 /r89049 /r89051 /r89061
/r89062 /r89063 /r89070 /r89071 /r89072 yaor_mdn/r89074 /r89075 /r89076 osnenl/r89079
/r89082 /r89083 /r89084 /r89085 /r89086 /r89087 eom-ylnphap/r89088 jedht/r89094 /r89099
/r89
[19:39:08] <codurr> /r89108 /r89110 /r89111 /r89112 /r89113 /r89114 /r89115 /r89116
/r89117 /r89118 /r89119 iwkiunta/r89120 /r89122 /r89123 onraa/r89128 /r89129 /r89134
/r89138 ^nmdoe/r89144 /r89145 /r89149 /r89150 algsinrtt/r89166 /r89176 /r89179 /r89180
sxemma/r89181 /r89182 /r89186 ptolsaiedn/r89191 /r89197 /r89204 /r89205 /r89206 /r89207
/r89208 /r89218 wadreoujdene/r89219 /r89223 /r89224 rtemo/r89225 /r89226 /r89227 /r89228
/r89230 /r89241 oribn/r89243 /r89244 /r89
[19:39:08] <codurr> kfoelokwrays/r89250 /r89251 /r89253 /r89254 awnurotkaot/r89258
/r89260 rnabsedi/r89261 /r89262 /r89263
--
Best regards,
Max Semenik ([[User:MaxSem]])