-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Use type and default option to confval in napoleon docs #13094
base: master
Are you sure you want to change the base?
Use type and default option to confval in napoleon docs #13094
Conversation
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.
Please use the new type and default options to confval
0ec4a68
to
5dccd13
Compare
doc/usage/extensions/napoleon.rst
Outdated
@@ -550,9 +575,10 @@ sure that "sphinx.ext.napoleon" is enabled in ``conf.py``:: | |||
.. versionadded:: 3.4 | |||
|
|||
.. confval:: napoleon_custom_sections | |||
:type: `Optional[Sequence[str | Sequence[str, str]]]` |
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.
Is it necessary to use Sequence
instead of list
or tuple
here? And should I use the |
, or nested Optional
?
napoleon_attr_annotations
I have updated the PR title to reflect changes not only to the documentation of the The previous title no longer accurately represented the scope of the updates. However, I have not changed the branch name, as doing so would close this PR. |
Inspired by the below code, I add the sphinx/doc/usage/configuration.rst Lines 2917 to 2919 in 230ccf2
|
|
Subject: Use type and default option to confval in napoleon docs
Feature or Bugfix
Purpose
Use type and default option to confval in napoleon docs.
Detail
None
Relates
None