Adds flake8 to our github actions checks so we can add it as a requirement for merging. This also adds the flake8-quotes plugin to make sure we're using double quotes everywhere. Since we requoted the whole codebase it seems that a few single quotes have snuck their way in because we had no automated checker.
Unfortunately there are still a number of error codes that we are ignoring that would be nice to fix, but are mostly due to legacy code so I don't feel like fixing them in this PR.
Adds flake8 to our github actions checks so we can add it as a requirement for merging. This also adds the
flake8-quotesplugin to make sure we're using double quotes everywhere. Since we requoted the whole codebase it seems that a few single quotes have snuck their way in because we had no automated checker.Unfortunately there are still a number of error codes that we are ignoring that would be nice to fix, but are mostly due to legacy code so I don't feel like fixing them in this PR.