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.