Added pre-commit #293
Added pre-commit #293
Conversation
|
It's a good idea. Could you also please adapt the CI / style part to use the pre-commit configuration? I want to make sure that there is only a single config for black and flake8 (and in near future also mypy). |
|
Good point, it should be working now (although this is the first time I'm messing with CI, so who knows lol) |
|
I don't think the failing test has anything to do with what I changed... |
|
I guess the problem is related to #284. I have recognized, that the requirements could be python version specified. We have observed a lot of issues around python 3.7 as it isn't supported by the latest numpy version, so i downgrade the numpy version to be compatible with 3.7. |
|
I have disabled the failing part of the CI for the moment. You could rebase and then we are able to merge |
|
Done. But not sure why that was necessary, there weren't any conflicts, were there? |
|
Again, a check unrelated to precommit is failing |
I just think that it's inconvenient to have style checks in CI but not locally so I added pre-commit (exactly matches the style checks in CI but excludes
mip/_version.py). Feel free to close this PR if there's a reason why you wouldn't want to have pre-commit.