Run flake8 for logic checks without style checks
Description
As agreed this morning.
I've tidied up most of the warnings that flake8 emits. There are still some for parameter annotations using undefined names that we'll need to decide what to do with.
How Has This Been Tested?
Running flake8 locally.
Types of changes
- Chore (non-code changes, e.g. comments, readme, grammar, etc...)
Checklist:
- My code follows the code style of this project.