Comment #2 on issue 1702 by smichr: Code Quality (was crlf) http://code.google.com/p/sympy/issues/detail?id=1702
Issue 1714's check on indentation would be good, too. (But I wouldn't want to see it enforced on continuation lines which are sometimes conveniently indented to show the structure of the long line.) Having gone over the docstrings a few times with spell checking/grammer pen in hand, it would be nice if that were a part of the review process as well. Sometimes the commit comments have problems with them, too, but once those are part of the commit history, they can't be changed. If docstring were on the check-off list before pushing, that would also keep that aspect of the project from getting behind. (But at the rate that Vinzent has been pushing in changes, perhaps that is less of a priority than the code/functionality?) -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings -- You received this message because you are subscribed to the Google Groups "sympy-issues" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/sympy-issues?hl=en.
