On 2013-01-31 10:58, Volker Braun wrote: > We definitely should have some lint-type tool that we put into the > commit hook to automatically show errors/warning.
But it's very important to check only newly added code, as there is so much code which doesn't conform to PEP 8. Also, I personally don't agree with everything that is said in PEP 8. I'd like to mention that in the release management script, there are a few things that I check for, such as: * no bare "except:" without exception class * no TABs * no use of AssertionError ("assert" is fine, but not "except AssertionError" for example) -- You received this message because you are subscribed to the Google Groups "sage-devel" group. To unsubscribe from this group and stop receiving emails from it, send an email to sage-devel+unsubscr...@googlegroups.com. To post to this group, send email to sage-devel@googlegroups.com. Visit this group at http://groups.google.com/group/sage-devel?hl=en. For more options, visit https://groups.google.com/groups/opt_out.