On 11/24/2012 07:48 PM, Volker Braun wrote:
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 trailing white space are disallowed. B) We do not care about trailing whitespace. Patch hunks that ONLY change trailing whitespace are disallowed.
How about C) which is between A) and B), and has been the current unofficial policy.
C) We care about trailing whitespace. No new patches should introduce trailing whitespace in modified lines. But patch hunks that ONLY change trailing whitespace are disallowed. It is of course encouraged to get rid of trailing whitespace in the code surrounding the lines your patch touches.
-- You received this message because you are subscribed to the Google Groups "sage-devel" group. To post to this group, send email to sage-devel@googlegroups.com. To unsubscribe from this group, send email to sage-devel+unsubscr...@googlegroups.com. Visit this group at http://groups.google.com/group/sage-devel?hl=en.
