-
Notifications
You must be signed in to change notification settings - Fork 67
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
Support Type Hints #94
Comments
Seconding this suggestion - we use |
yea, there was a request for _Matcher and _Context to be public classes. I don't really like that mypy forces that, there's no reason for them to be public, but it's not a big deal in practice. |
@jamielennox @jonadaly can you try out my PR? I've been using those hints for a while but always with the same pattern so it's worth running it against other pieces of code before submitting. Also I've considered checking them with |
I've just run your PR against some code I've got (non-public, so can't share links), and works perfectly :) |
Your PR works for me! |
Going to mark this closed by #161. If it needs to be enhanced it can be done in an incremental way. Thanks for the input. |
Released as 1.9.0 |
Awesome. Upgraded my project, deleted a bunch of internal hacky stubs :) |
Thanks @jamielennox for the release, and the quick patch! It's working like a charm for us as well. |
We should make an attempt to provide python type hinting support. It will need to continue to support python2.7 and I would like to figure out how it can be unit tested through mypy or something.
The text was updated successfully, but these errors were encountered: