Replace complicated quality checks with flake8
This MR effectively reverts !63 (merged) in favour of something more like !59 (closed), which is much simpler, using flake8
for the entire analysis.
This also expands the analysis to include docstrings and security issues.