Introduce `pre-commit` for static code analysis

py-stockfish / stockfish

Integrates the Stockfish chess engine with Python (Official fork)

https://py-stockfish.github.io/stockfish/

MIT License

29 stars 8 forks source link

Closed kieferro closed 11 months ago

kieferro commented 11 months ago

This PR adds pre-commit to the techstack:

The used tools are black and mypy as before. I have also added a few other simple ones like trailing-whitespace. We can expand this in the future. For example with pylint, but I think a separate PR is better for that because we will have to make some changes to satisfy pylint.
I have activated the CI which has already run on this PR and will run on the master branch once we push to it

github-actions[bot] commented 11 months ago

The coverage rate went from 94.94% to 94.94% :arrow_right: The branch rate is 89%.

None of the new lines are part of the tested code. Therefore, there is no coverage data about them.