-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
Improve documentation #7813
Improve documentation #7813
Conversation
Codecov Report
@@ Coverage Diff @@
## master #7813 +/- ##
==========================================
- Coverage 65.89% 65.89% -0.01%
==========================================
Files 459 459
Lines 22011 22016 +5
Branches 2417 2417
==========================================
+ Hits 14505 14507 +2
- Misses 7385 7388 +3
Partials 121 121
Continue to review full report at Codecov.
|
@@ -47,6 +47,7 @@ | |||
extensions = [ | |||
'sphinx.ext.autodoc', | |||
'sphinx.ext.viewcode', | |||
'sphinx_autodoc_typehints', |
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.
This is sweet.
I'll go ahead and merge this right away, I have another PR that is partially dependent on this. |
CATEGORY
Choose one
SUMMARY
I was working on something unrelated and noticed the need to change the following:
current_user_id
andcurrent_username
jinja functionsSphinx
deps and add support for type annotations (added dependency on MIT licensedsphinx_autodoc_typehints
)TEST PLAN
Tested locally
BEFORE
AFTER
REVIEWERS
@mistercrunch @john-bodley