-
Notifications
You must be signed in to change notification settings - Fork 148
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
Deprecate when
parameter of naturaldelta
#248
Conversation
Codecov Report
@@ Coverage Diff @@
## master #248 +/- ##
==========================================
+ Coverage 97.08% 97.10% +0.02%
==========================================
Files 10 10
Lines 685 692 +7
==========================================
+ Hits 665 672 +7
Misses 20 20
Flags with carried forward coverage won't be shown. Click here to find out more.
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.
Thanks, couple of little nits.
Co-authored-by: Hugo van Kemenade <[email protected]>
Co-authored-by: Hugo van Kemenade <[email protected]>
I also want to ask if you actually use Sphinx for you documentation. (Those directives for deprecation are for Sphinx). I seems like the readthedocs page for this repository hasn't been updated in some time. https://humanize.readthedocs.io/en/latest/# Oh, the docs are hosted at python-humanize.readthedocs.io, but I'm still not sure if y'all are using Sphinx. |
Yep, docs are at https://python-humanize.readthedocs.io which uses MkDocs not Sphinx so it'll just have to be written as plain text. To build and view docs:
|
parameter of
naturaldelta`
parameter of
naturaldelta`when
parameter of naturaldelta
Thank you! |
Related to #247, but does not close.
Adds deprecation warnings for
naturaldelta(..., when)
because this parameter does not make sense for a timedelta.Didn't add test because it seems silly to test for the presence of deprecation warnings that will only be around until the next release.