-
-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix mypy #358
Fix mypy #358
Conversation
I think we should just exclude this file from mypy, it's not typed at all. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, failures are unrelated, let's fix them in another PR.
Wait, pre-commit is still failing? It means mypy is still analysing this file? Also, we don't need from pkg_resources import parse_version # type: ignore |
Yes, I saw that...
The 2 first commits that I reverted fixed mypy for that file. |
5dcdf97
to
a024452
Compare
a024452
to
6889d27
Compare
Thanks @davidbrochart. Should we remove
from pyproject.toml to avoid confusion, or is this useful for local run ? |
Yes I think it's still useful outside of pre-commit. |
This PR should fix mypy which fails with the
bump_version.py
file.