Skip to content
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

Jump to bottom

An index entry with parens was registered for py:method directive with :property: option #9697

Closed
tk0miya opened this issue Oct 3, 2021 · 0 comments
Closed

An index entry with parens was registered for py:method directive with :property: option #9697

tk0miya opened this issue Oct 3, 2021 · 0 comments
Labels
domains:py type:bug
Milestone
4.3.0

Comments

@tk0miya
Copy link
Member

tk0miya commented Oct 3, 2021

Describe the bug

An index entry with parens was registered for py:method directive with :property: option. It should not have parens.

How to Reproduce

# index.rst

.. py:method:: Foo.bar
   :property:

.. py:property:: Foo.baz

Expected behavior

An index entry for the property should not have parens.

Your project

N/A

Screenshots

スクリーンショット 2021-10-03 13 00 53

OS

Mac

Python version

3.9.6

Sphinx version

HEAD of 4.x

Sphinx extensions

No response

Extra tools

No response

Additional context

No response
@tk0miya tk0miya added this to the 4.3.0 milestone Oct 3, 2021
@tk0miya tk0miya mentioned this issue Oct 3, 2021
Merged
@tk0miya tk0miya closed this as completed in b920418 Oct 9, 2021
tk0miya added a commit that referenced this issue Oct 9, 2021
@tk0miya
Merge pull request #9698 from tk0miya/9697_property_with_parens
7da94ee 
Fix #9697: An index entry with parens was registered for py:method
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
domains:py type:bug
Projects
None yet
Milestone
4.3.0
Development

No branches or pull requests
1 participant
@tk0miya