-
Notifications
You must be signed in to change notification settings - Fork 318
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
Clean up deprecations #650
Conversation
Codecov Report
@@ Coverage Diff @@
## main #650 +/- ##
==========================================
+ Coverage 77.79% 77.96% +0.16%
==========================================
Files 110 110
Lines 10465 10405 -60
Branches 1419 1400 -19
==========================================
- Hits 8141 8112 -29
+ Misses 1923 1906 -17
+ Partials 401 387 -14
Continue to review full report at Codecov.
|
@@ -232,7 +232,7 @@ def _headers_default(self): | |||
default_value=None, | |||
allow_none=True, | |||
config=True, | |||
help="""The authorization token used in the HTTP headers. The header will be formatted as: | |||
help="""The authorization token used in the HTTP headers. The header will be formatted as:: |
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.
As this extra colon added on purpose?
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.
Yes, this docstring is copied directly into an RST file, and was causing warnings without the extra colon.
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, @blink1073. One nitpicky comment, but otherwise looks good.
Fixes #627
Fixes #644
Drops Python 3.6, which is EOL
This requires a minor version release.