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.


Reply via email to