On 11/25/2012 12:02 AM, Volker Braun wrote:
On Saturday, November 24, 2012 2:11:44 PM UTC, P Purkayastha wrote:But, we do want our code to look "good" and be less frustrating to hack around, don't we? But thats precisely my point, trailing whitespace is by definition not visible so it doesn't change whether your code "looks" good or not.
Yes. I also mentioned the "less frustrating" part. It doesn't behave as it looks.
I had a quick look at some of the bigger projects around, and almost all of them have a *strict* policy of not allowing trailing whitespace - numpy/scipy, mozilla, linux, matplotlib, etc. Surely, lot of people care about not having this in their code. In Sage, the current (unofficial) policy is way more tolerant of this behavior, and what I proposed is to simply *encourage* people to not put trailing whitespace in their new edits. You don't lose much except the fact that the patchbot will refuse to doctest your patch. I don't know if the release manager will refuse to merge your patches if they are not tested by the patchbot.
-- 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.
