Exactly. This is why I put that warning in there. I was thinking of pylint. There's supposed to be a way to customize pylint, so that you can disable the useless warnings. Otherwise, I would just stick to pyflakes.
And note that sometimes the tests will still pass, e.g., when a variable is not defined because that particular code path is not covered by the tests. You can see what lines are covered by using the ./bin/coverage_report.py script (I think we also have some tasks for this too). Aaron Meurer On Mon, Nov 21, 2011 at 3:01 PM, Matt Habel <habel...@gmail.com> wrote: > Pylint is giving me a buttload of errors that don't really seem like > errors. The tests run and pass fine, so I really can't figure out what > pylint is complaining about. Maybe I set it up wrong? > > On Nov 21, 4:54 pm, Aaron Meurer <asmeu...@gmail.com> wrote: >> The functions module is fine. I just tried it, and I think they can >> all be fixed. The only exception would be if pyflakes has made a >> mistake and things that something is wrong but really isn't (this can >> happen sometimes due to the dynamic nature of Python). >> >> I think it's actually pylint, not pyflakes, that I was thinking of >> regarding that comment. I will modify the task description to make >> this clear. Note that pylint gives way more output, but a lot of it >> is things like "Invalid name "z" (should match >> [a-z_][a-z0-9_]{2,30}$)", but short or long names like that are >> actually fine. You can use pylint or pyflakes to do this task (but if >> you choose pylint, ignore the invalid name and "too many..." >> warnings). >> >> Aaron Meurer >> >> >> >> >> >> >> >> On Mon, Nov 21, 2011 at 2:20 PM, Matt Habel <habel...@gmail.com> wrote: >> > Hi there mentors and others of this project, I am just following up on >> > my task claim from earlier today. As stated, I need to find a good >> > module to work on, would the functions module be okay? >> >> > Also, what kind of errors are skippable? >> >> > Thanks >> >> > -- >> > You received this message because you are subscribed to the Google Groups >> > "sympy" group. >> > To post to this group, send email to sympy@googlegroups.com. >> > To unsubscribe from this group, send email to >> > sympy+unsubscr...@googlegroups.com. >> > For more options, visit this group >> > athttp://groups.google.com/group/sympy?hl=en. > > -- > You received this message because you are subscribed to the Google Groups > "sympy" group. > To post to this group, send email to sympy@googlegroups.com. > To unsubscribe from this group, send email to > sympy+unsubscr...@googlegroups.com. > For more options, visit this group at > http://groups.google.com/group/sympy?hl=en. > > -- You received this message because you are subscribed to the Google Groups "sympy" group. To post to this group, send email to sympy@googlegroups.com. To unsubscribe from this group, send email to sympy+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/sympy?hl=en.