-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add flake8 (with dlint) tests to CI #1195
Comments
Hey, glad to see this change! Regarding false-positives from Generally I find running all the Regarding Dlint's other findings:
If you don't believe these are security issues, then I'd ignore them inline. Happy to help here if you'd like! |
#1194 showed that adopting properly configured static analysis might help prevent some serious bugs. There are lots of style warnings and false-positives currently when running flake8. Some are fixable, but some warnings could also simply be disabled.
The text was updated successfully, but these errors were encountered: