Remove some uses of overload - #665
Conversation
e748deb to
4863d53
Compare
|
Yeah, I had a bunch of errors, too. I'm fixing them... |
|
What the heck? I think I managed to crash mypy!? Those Travis build results are screwed up... |
There was a problem hiding this comment.
I think you're missing the return type here for Callable. I think this is causing the failing tests.
|
Of course, mypy shouldn't crash in this way as a result of a missing argument. I've made issue #673 for this. |
|
@spkersten Thanks! I didn't realize that! What exactly do you mean by a default argument for |
|
Wow, a single PR causing two mypy crashes! You found some code paths that don't have good test coverage. Narrowed down the second crash: #674 Life is busy currently, but I'll try to find some time this week to debug the issue. |
|
Fixed #674. Started another build -- hopefully it passes now. |
|
@kirbyfan64 You van ignore the comment about default for string. I was misreading the diff and thought that one of the original signatures didn't have the string argument. |
|
Looks good! Thanks for the PR! |
3 participants
This fixes #432 and removes some other uses of
@overloadthat were pretty simple.