Skip to content
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

MAINT: unpin mypy #900

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tylerjereddy
Copy link
Collaborator

  • mypy 1.0.0 was released on Feb. 6/2023, and we recently pinned it down to older versions while working on more important things because we had a few new type hinting issues pop up

  • fix the type hinting issues, which were actually just ignore directives on try/except blocks that are no longer needed, so that's a clear win I think

  • I doubt it matters much for us, but mypy 1.0.0 can be ~40 % faster on large codebases:
    https://mypy-lang.blogspot.com/2023/02/mypy-10-released.html

* `mypy` `1.0.0` was released on Feb. 6/2023, and we recently pinned
it down to older versions while working on more important things because
we had a few new type hinting issues pop up

* fix the type hinting issues, which were actually just ignore
directives on `try/except` blocks that are no longer needed, so that's
a clear win I think

* I doubt it matters much for us, but `mypy 1.0.0` can be ~40 %
faster on large codebases:
https://mypy-lang.blogspot.com/2023/02/mypy-10-released.html
@tylerjereddy tylerjereddy added CI continuous integration maintenance labels Feb 14, 2023
@tylerjereddy
Copy link
Collaborator Author

Ah, confusingly the issue does still exist on mypy 1.0.0: python/mypy#13914 (comment)

It just didn't show up when I tested with Python 3.10 locally, which uses a different code path.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI continuous integration maintenance pydarshan
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant