On Apr 15, 2010, at 2:48 PM, Nathan O'Treally wrote:

I'm not happy with "brute-force" converting *any* tab to space(s), and
it would be better to have a tool that (conditionally) does this
rather than supplying patches to lots of files converted by Emacs. The
same tool could be used just to check for "illegal" tabs. As I
understand this, these are *tabs in leading whitespace*.

Trailing whitespace could be removed. (Currently trailing tabs are
converted to many trailing spaces.)

I've also come across files that do not end with newline...

Opinions?

I think we should do this once, after a tool is in place to make sure these things don't get in anymore.

A big +1 to sage -t failing on files that have tabs, lack newlines, or whatever else the style guide enforces. I've also thought it would be useful to have a trac plugin that points out such things (and it could report coverage changes as well).

- Robert

--
To post to this group, send an email to sage-devel@googlegroups.com
To unsubscribe from this group, send an email to 
sage-devel+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/sage-devel
URL: http://www.sagemath.org

To unsubscribe, reply using "remove me" as the subject.

Reply via email to