-
Notifications
You must be signed in to change notification settings - Fork 47
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
Make _magicgui.pyi
stubs
#126
Conversation
Codecov Report
@@ Coverage Diff @@
## master #126 +/- ##
==========================================
- Coverage 90.55% 90.51% -0.04%
==========================================
Files 25 25
Lines 2445 2435 -10
==========================================
- Hits 2214 2204 -10
Misses 231 231
Continue to review full report at Codecov.
|
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.
Wow, you figured it out??? What was the issue?
app: AppRef = None, | ||
**param_options: dict, | ||
) -> Callable[[Callable[..., _R]], MainFunctionGui[_R]]: ... | ||
@overload # noqa: E302 |
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.
Why are you squishing this so much?
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.
that's what black does to pyi files! 😂
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.
What was the issue?
turns out it was just me being dumb. I was trying to test it interactively in the _magicgui.py
file. Now that I have the tests in typesafety
I realized they were working even though it didn't seem to be working inside of _magicgui.py
... so I just tried somewhere else and it worked!
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.
that's what black does to pyi files! 😂
🤢
The type annotations in
_magicgui.py
got a little unsightly in #117. This puts all of the overloads and other typing stuff into_magicgui.pyi
. Tests for typing annotations were added (in thetypesafety
folder) in #117 and are still passing here.