Skip to content

gh-101100: Fix sphinx warnings in library/xml.sax.handler.rst #136612

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

LamentXU123
Copy link
Contributor

@LamentXU123 LamentXU123 commented Jul 13, 2025

There are two warnings:

C:\Users\admin\Downloads\cpython-main\Doc\library\xml.sax.handler.rst:249: WARNING: py:class reference target not found: xml.sax.xmlreader.Attributes [ref.class]
C:\Users\admin\Downloads\cpython-main\Doc\library\xml.sax.handler.rst:271: WARNING: py:class reference target not found: xml.sax.xmlreader.AttributesNS [ref.class]

This warning happens in many doc of xml (xml.sax.reader, xml.sax.handler, xml.sax) in the same reason. After staring the doc for a while, I think the original doc want to refer to xml.sax.xmlreader.AttributesImpl and xml.sax.xmlreader.AttributesNSImpl as an implemention of attributes-interface. This doc fix it to the correct ref.

If this is approved, I will fix up those occurrence in other docs, an example of this in xml.sax is as below to double check.

image

📚 Documentation preview 📚: https://cpython-previews--136612.org.readthedocs.build/

@LamentXU123 LamentXU123 changed the title gh-101100: Fix sphinx warnings in library/xml.sax.rst gh-101100: Fix sphinx warnings in library/xml.sax.handler.rst Jul 14, 2025
@LamentXU123 LamentXU123 marked this pull request as draft July 14, 2025 15:21
@LamentXU123
Copy link
Contributor Author

Fixed to:
image

@LamentXU123 LamentXU123 marked this pull request as ready for review July 14, 2025 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review docs Documentation in the Doc dir skip news
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

2 participants