Closed cclauss closed 4 years ago
@cclauss thanks so much for this. This project has been in suspended animation for some time but I will be getting things moving again in short order.
@cclauss Thank you very much for this. I have now linted this entire project with black, codespell, and flake8. I made some modifications to the Github action you created here but the spirit remains the same.
Among other things I have made the action fail if any of these three tools find problems. At the moment, these checks still do not pass. This is because there are a handful of remaining problems found by black and flake8. They fall into two categories:
gi
library requires some funky import behavior. I am hoping to migrate this project away from GTK at some point soon so that should solve these problems.My hope is that we can get these remaining problems cleaned up before long and then begin enforcing this action for all PRs.
Demonstrating why we need #3 or similar. Five files with Python syntax errors.
Output: https://github.com/cclauss/sobot-rimulator/actions
flake8 testing of https://github.com/nmccrea/sobot-rimulator on Python 3.8.3
$ flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
https://flake8.pycqa.org/en/latest/user/error-codes.html
On the flake8 test selection, this PR does not focus on "style violations" (the majority of flake8 error codes that psf/black can autocorrect). Instead these tests are focus on runtime safety and correctness: