Introduces black (code formatting), pylint (static code analysis), and isort (import sorting).
Initial code formatting applied.
Current linting issues have had an initial ignore added. This PR does not address the issues, just introduces the minimum change set to perform (and enforce) styling and linting. This issues will need to be addressed separately.
PRs must now also pass linting, and static analysis checks.
Why
Enforces proper software engineering best practices.
If there is strong preferences for a different suite if tools, please let me know. These are the ones I'm familiar with and use regularly but there may be more widely understood and accepted combos.
What
No new business logic introduced.
Introduces black (code formatting), pylint (static code analysis), and isort (import sorting).
Initial code formatting applied.
Current linting issues have had an initial
ignore
added. This PR does not address the issues, just introduces the minimum change set to perform (and enforce) styling and linting. This issues will need to be addressed separately.PRs must now also pass linting, and static analysis checks.
Why
How was this tested
Notes