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
DM-36108: move Ellipsis typing workaround from daf_butler #137
Conversation
My in-editor mypy has long complained about this, even though the command-line one doesn't seem bothered by it. It's easy enough to fix.
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.
Looks okay but can you have a test that imports the code so that we know it will import?
a7206ef
to
8e6c4c7
Compare
Codecov Report
@@ Coverage Diff @@
## main #137 +/- ##
==========================================
- Coverage 92.24% 92.12% -0.13%
==========================================
Files 42 44 +2
Lines 2709 2729 +20
==========================================
+ Hits 2499 2514 +15
- Misses 210 215 +5
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
8e6c4c7
to
fd0e696
Compare
fd0e696
to
c28f55c
Compare
|
||
from typing import TYPE_CHECKING | ||
|
||
if TYPE_CHECKING: |
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.
It's expected for codecov to complain about this block, because by design it's only ever seen by MyPy and other type-checkers: we don't actually want it to run in tests.
Checklist
doc/changes