psychopy.org | Reference | Downloads | Github

Static code analysis

#1

Hi,

I found this online tool that does static code analysis for python (lgtm.com).
I run psychopy with it and it had some things to say:
https://lgtm.com/projects/g/psychopy/psychopy/alerts/?mode=tree&severity=error

Just wanted to share if any of the core developers are interested.
Also to check if you would welcome PRs that are from “error” of code analysis and not user reported.

Boaz

#2

Definitely looks interesting. Several of those look real and some look to me like potential false alarms.

For instance:

  • the Raise errors look like real errors.
  • one of the Wrong number of arguments for format errors looks like a false alarm

Would be good to see some pull requests fixing these, but maybe a few at a time (in logical batches).