[sage-devel] Disable the "trailing whitespace" patchbot plugin

2012-11-23 Thread Volker Braun
I'd like to disable the trailing whitespace plugin for the patchbot, or at least ignore its output in the scoring. All it does is a) discourage people from reviewing patches ("the light is not green") b) misguides people writing patches into making unnecessary whitespace changes ("trailing whi

Re: [sage-devel] Disable the "trailing whitespace" patchbot plugin

2012-11-23 Thread Robert Bradshaw
On Fri, Nov 23, 2012 at 12:23 PM, Volker Braun wrote: > I'd like to disable the trailing whitespace plugin for the patchbot, or at > least ignore its output in the scoring. All it does is > > a) discourage people from reviewing patches ("the light is not green") People *should* look at blue-blurb

Re: [sage-devel] Disable the "trailing whitespace" patchbot plugin

2012-11-24 Thread Volker Braun
On Saturday, November 24, 2012 7:41:20 AM UTC, Robert Bradshaw wrote: > Anyone want to go first in formulating a policy? The two obvious possibilities that are easily verifyable for a computer are: A) No trailing whitespace is allowed anywhere. Strip the Sage library. Patches that introduce tr

Re: [sage-devel] Disable the "trailing whitespace" patchbot plugin

2012-11-24 Thread Ivan Andrus
On Nov 24, 2012, at 3:38 PM, Volker Braun wrote: > On Saturday, November 24, 2012 2:11:54 PM UTC, Ivan Andrus wrote: > I disagree. Perhaps I am alone in this, but I often (used to) move to the > end of the line, go back a few characters (to get inside parenthesis or > something), and start typ