-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Restore 'noindex' support for JS, Python domains #12844
Conversation
Signed-off-by: Stephen Finucane <[email protected]> Fixes: e439c6f (Ensure that old-style object description options are respected (sphinx-doc#12620)) Closes: sphinx-doc#12843
Nope, we don't have backports (too much work and not enough hands...) |
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.
Are these the only parts in the code base that require this? and could you perhaps add a test for that?
I could, but I'm away for the next week and a bit so I'll be a while. I also don't know if it's essential given this is effectively continuing what was done in e439c6f through to completion and that didn't have tests. In addition, this is for a feature that will be deprecated soon and I have a reproducer linked in the bug that proves this fixes the issue, albeit manually. So maybe we can skirt around without one here and just get this in? |
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.
I'll try and add a test later today, but I'm on a flight at some point so may not be that productive.
A
Thanks 🙏 |
Feature or Bugfix
Purpose
Fixes: e439c6f (Ensure that old-style object description options are respected (#12620))
Closes: #12843
Detail
no-index
ignored for:py:module:
#12843Relates
None.