-
Notifications
You must be signed in to change notification settings - Fork 108
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
Type annotate the temporal module #604
base: typecheck-apiformatting
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## typecheck-apiformatting #604 +/- ##
===========================================================
+ Coverage 65.44% 65.48% +0.04%
===========================================================
Files 38 38
Lines 3128 3132 +4
Branches 601 601
===========================================================
+ Hits 2047 2051 +4
Misses 990 990
Partials 91 91 ☔ View full report in Codecov by Sentry. |
7f92261
to
e805f01
Compare
start_time: string, datetime.time, None | ||
end_time: string, datetime.time, None |
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 remove the start_time
and end_time
parameters? Shouldn't we just remove the type?
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.
Since they're part of the function signature, they will be documented automatically; we only need to mention things in the docstring that aren't already represented in the function signature. One less thing to keep in sync :)
e805f01
to
7b57037
Compare
ca3f553
to
3da94ef
Compare
Important
To be merged only after merging #598 and then rebasing
Annotating this module was pretty quick! No typeguards or ignores needed.