Enable basic type checking and linting via pre-commit #49
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a very early attempt to enable standard linting for consistency in line length, variable names, and annotations.
I've also updated this repo to use a consistent version of python across all of our containers and packages and tests.
I've had to add in a few workarounds for some code that was nowhere near passing lint and type checking:
#noqa
comments on lines that are either low-priority or extremely difficult to fix.platformics/api/relay
andplatformics/thirdparty
was very far away from being able to be type-checked well, so those are ignored in our mypy configurations at the moment.