-
Notifications
You must be signed in to change notification settings - Fork 11
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
More typing #76
More typing #76
Conversation
Just realized args and kwargs can probably be annotated as |
Aight, it doesn't seem to work... But i see that |
I just added an ignore for that, it's mypy getting picky about a variable that's only used internally to the function. If you rebase it should be working now. |
Just in case, this is ready for review/merge on my end |
Oops, seems like mypy didn't like
So... im thinking about ordering like:
|
I think that can work, it still bundles the code at the end. If you can't find a way that works with code at the end, we can settle for what does work. |
Looks like the latest commit passes. Ready to merge? |
Yeah! |
Thanks! |
I have just noticed i missed these on my first PR... Sorry about the extra noise
Note that the
_question
implementation is a bit hacky, but seems to work completely fine with mypy :)