Skip to content

gh-151944: Fix Sphinx reference warnings in urllib.{parse,request}.rst#152162

Open
lpyu001 wants to merge 4 commits into
python:mainfrom
lpyu001:fix-Sphinx-reference-warnings-in-urllib
Open

gh-151944: Fix Sphinx reference warnings in urllib.{parse,request}.rst#152162
lpyu001 wants to merge 4 commits into
python:mainfrom
lpyu001:fix-Sphinx-reference-warnings-in-urllib

Conversation

@lpyu001

@lpyu001 lpyu001 commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

@bedevere-app bedevere-app Bot added docs Documentation in the Doc dir skip news awaiting review labels Jun 25, 2026
@github-project-automation github-project-automation Bot moved this to Todo in Docs PRs Jun 25, 2026
@lpyu001 lpyu001 changed the title fix-Sphinx-reference-warnings-in-urllib.{parse,request}.rst gh-151944:fix-Sphinx-reference-warnings-in-urllib.{parse,request}.rst Jun 25, 2026
@lpyu001 lpyu001 changed the title gh-151944:fix-Sphinx-reference-warnings-in-urllib.{parse,request}.rst gh-151944: Fix Sphinx reference warnings in urllib.{parse,request}.rst Jun 25, 2026
@read-the-docs-community

read-the-docs-community Bot commented Jun 25, 2026

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #33411203 | 📁 Comparing 26de888 against main (a6c2d4a)

  🔍 Preview build  

49 files changed · ± 48 modified · - 1 deleted

± Modified

- Deleted

@StanFromIreland StanFromIreland changed the title gh-151944: Fix Sphinx reference warnings in urllib.{parse,request}.rst gh-151944: Fix Sphinx reference warnings in urllib.{parse,request}.rst Jun 25, 2026
Comment thread Doc/library/urllib.parse.rst Outdated
If the *allow_fragments* argument is false, fragment identifiers are not
recognized. Instead, they are parsed as part of the path
or query component, and :attr:`fragment` is set to ``None`` or the empty
or query component, and :attr:`fragment <SplitResult.fragment>` is set to

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
or query component, and :attr:`fragment <SplitResult.fragment>` is set to
or query component, and :attr:`~SplitResult.fragment` is set to

Comment thread Doc/library/urllib.parse.rst Outdated
.. [1] Depending on the value of the *missing_as_none* argument.

Reading the :attr:`port` attribute will raise a :exc:`ValueError` if
Reading the :attr:`port <SplitResult.port>` attribute will raise a

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same for all of these.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I’ve addressed it,thanks

Comment thread Doc/library/urllib.request.rst
@hugovk

hugovk commented Jul 2, 2026

Copy link
Copy Markdown
Member

Please also update the headers to be sentence case (https://devguide.python.org/documentation/style-guide/#capitalization):

image image

@lpyu001

lpyu001 commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

Thanks, I’ve addressed your comment. @hugovk

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.

3 participants