Updates:
Summary: Code Quality (was crlf)
Cc: asmeurer ondrej.certik
Comment #1 on issue 1702 by smichr: Code Quality (was crlf)
http://code.google.com/p/sympy/issues/detail?id=1702
There are a few things that I would like to see in the testing process to
verify a patch:
1) a check that no coverage has been lost. This takes a lot of time on my
machine...maybe this could be the last test run before considering a push.
If a
change has been made that uncovers some code that is a good opportunity to
get a real
test (rather than an incidental test). Some modules are way behind where
they should
be. Perhaps we should have a "increase-the-coverage" day.
2) a check that there are no inline logic statements, like
if: foo
else: bar
You can't tell if these have been covered or not in the coverage reports.
--
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.
