Audit flake8 CI errors vs. warnings #2
Description
#1 mentions that the flake8 in workflows/lint.yml may be overly permissive. This could result in bad code getting in.
- evaluate error on F401 for unused imports
- evaluate error on F841 for local variable assigned but unused
Note that pre-commit config already prohibits both of these.