On Dec 4, 2012, at 9:46 PM, Daniel Friesen daniel@nadir-seen-fire.com wrote:
On Tue, 04 Dec 2012 12:37:02 -0800, Chad innocentkiller@gmail.com wrote:
On Tue, Dec 4, 2012 at 3:27 PM, Chad innocentkiller@gmail.com wrote:
On Tue, Dec 4, 2012 at 3:24 PM, Tyler Romeo tylerromeo@gmail.com wrote:
Don't we have some sort of policy about an individual merging commits that he/she uploaded?
Yes. We've been over this a dozen times--if you're on a repository that has multiple maintainers (ie: you're not the only one, so you're always self-merging), you should almost never merge your own code unless you're fixing an immediate problem (site outage, sytax errors).
In fact, I'm tired of repeating this problem, so I started a change to actually enforce this policy[0]. We'll probably need to tweak it further to allow for the exceptions we actually want. Review welcome.
-Chad
Doesn't TWN's bot self-review? Might need to add an exception for that before merging.
I'm not sure in which part of the flow rules.pl is applied but maybe it can be enforced the other way around?
Instead of restricting Submit, restrict CR scores. Submission in turn only has to be restricted to CR+2.
But yeah, we need to either whitelist L10n-bot from this restriction or make those commits auto-merge in a different way.
-- Krinkle