heisenbridge icon indicating copy to clipboard operation
heisenbridge copied to clipboard

CI: add Flake8 and Mypy checks

Open piegamesde opened this issue 3 years ago • 5 comments

What isn't checked automatically isn't worth the effort.

piegamesde avatar May 03 '21 11:05 piegamesde

Flake8 is being run in the pre-commit task which is part of the CI. Mypy isn't enabled until it passes.

hifi avatar May 03 '21 11:05 hifi

Oh, I totally missed the pre-commit hook, I'll remove Flake8 then. If you want, I can reopen the pull request against the feature/make-mypy-happy so that you merge that one once the check passes.

piegamesde avatar May 03 '21 11:05 piegamesde

The feature branch is badly outdated and probably just needs to be redone completely, unfortunately. It was also a bad attempt at fixing it.

hifi avatar May 03 '21 11:05 hifi

Do you want me to give a second try at fixing all the type annotations? Also, why was it a bad attempt?

piegamesde avatar May 03 '21 12:05 piegamesde

I was adding more classes to fix typing issues, @tetov did a second run in some of his own branches which was better AFAIK.

Edit: and yes of course if you have time you are more than welcome to try :bow:

hifi avatar May 03 '21 12:05 hifi