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.

assigned to @duncanmmacleod

added 1 commit

• 7b2a76e8 - ci: replace complicated quality checks with flake8

Compare with previous version

added 1 commit

• a2e69e36 - ci: replace complicated quality checks with flake8

Compare with previous version

added 1 commit

• f4c36eb7 - ci: replace complicated quality checks with flake8

Compare with previous version

added 1 commit

• b055977c - ci: replace complicated quality checks with flake8

Compare with previous version

added 1 commit

• ea682006 - ci: replace complicated quality checks with flake8

Compare with previous version

merged

mentioned in commit 937dc8bc