GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-101100: Resolve reference warnings in reference/ by AA-Turner · Pull Request #138418 · python/cpython

Conversation

@AA-Turner

@AA-Turner AA-Turner commented Sep 3, 2025 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member

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

  • Issue: Fix all Sphinx reference warnings in the documentation #101100

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@AA-Turner
Resolve reference warnings in reference/
01a4bee
@AA-Turner AA-Turner added needs backport to 3.13 only security fixes needs backport to 3.14 bugs and security fixes labels Sep 3, 2025
@bedevere-app bedevere-app Bot added docs Documentation in the Doc dir skip news labels Sep 3, 2025
@github-project-automation github-project-automation Bot added this to Docs PRs Sep 3, 2025
@github-project-automation github-project-automation Bot moved this to Todo in Docs PRs Sep 3, 2025
@AA-Turner AA-Turner mentioned this pull request Sep 3, 2025
Add class directives for xml.dom #138419
Closed
@AA-Turner AA-Turner changed the title Resolve reference warnings in reference/ GH-101100: Resolve reference warnings in reference/ Sep 3, 2025
@bedevere-app bedevere-app Bot mentioned this pull request Sep 3, 2025
Fix all Sphinx reference warnings in the documentation #101100
Closed
@AA-Turner
AA-Turner marked this pull request as ready for review September 3, 2025 00:27
@AA-Turner
AA-Turner requested review from hugovk and willingc as code owners September 3, 2025 00:27
@bedevere-app bedevere-app Bot added the awaiting core review label Sep 3, 2025
@AA-Turner
AA-Turner marked this pull request as draft September 3, 2025 00:27
@bedevere-app bedevere-app Bot removed the awaiting core review label Sep 3, 2025
@AA-Turner
Use :ref: for NodeList
f31c2f9
@AA-Turner
AA-Turner marked this pull request as ready for review September 3, 2025 20:46
@bedevere-app bedevere-app Bot added the awaiting core review label Sep 3, 2025
AA-Turner added 2 commits September 3, 2025 21:47
@AA-Turner
Merge branch 'main' into docs/refwarn/datamodel
c62a275
# Conflicts:
#	Doc/tools/.nitignore
@AA-Turner
Merge branch 'main' into docs/refwarn/datamodel
08255a1
@hugovk hugovk changed the title GH-101100: Resolve reference warnings in reference/ gh-101100: Resolve reference warnings in reference/ Sep 8, 2025
@hugovk

hugovk commented Sep 8, 2025

Copy link
Copy Markdown
Member

@hugovk changed the title
GH-101100: Resolve reference warnings in reference/
gh-101100: Resolve reference warnings in reference/

The convention is gh for issues and GH for PRs :)

Here “gh-12345” is the GitHub issue number, and “GH-NNNN” is the number of the original pull request.

https://devguide.python.org/core-team/committing/#backporting-changes-to-an-older-version

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

hugovk
hugovk reviewed Sep 8, 2025
View reviewed changes
Comment thread Doc/reference/datamodel.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.

@AA-Turner @hugovk
Update Doc/reference/datamodel.rst
e5f1a5b
Co-authored-by: Hugo van Kemenade <[email protected]>
@AA-Turner
AA-Turner enabled auto-merge (squash) September 8, 2025 19:25
@AA-Turner
AA-Turner merged commit 6831634 into python:main Sep 8, 2025
@github-project-automation github-project-automation Bot moved this from Todo to Done in Docs PRs Sep 8, 2025
@miss-islington-app

miss-islington-app Bot commented Sep 8, 2025

Copy link
Copy Markdown

Thanks @AA-Turner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-app bedevere-app Bot removed the awaiting core review label Sep 8, 2025
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 8, 2025
@AA-Turner @miss-islington
pythongh-101100: Resolve reference warnings in reference/ (pythonGH-1…
2d635ec
…38418)

(cherry picked from commit 6831634)

Co-authored-by: Adam Turner <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 8, 2025
@AA-Turner @miss-islington
pythongh-101100: Resolve reference warnings in reference/ (pythonGH-1…
586b25e
…38418)

(cherry picked from commit 6831634)

Co-authored-by: Adam Turner <[email protected]>
@bedevere-app

bedevere-app Bot commented Sep 8, 2025

Copy link
Copy Markdown

GH-138680 is a backport of this pull request to the 3.14 branch.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Sep 8, 2025
@bedevere-app

bedevere-app Bot commented Sep 8, 2025

Copy link
Copy Markdown

GH-138681 is a backport of this pull request to the 3.13 branch.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@bedevere-app bedevere-app Bot removed the needs backport to 3.13 only security fixes label Sep 8, 2025
@AA-Turner
AA-Turner deleted the docs/refwarn/datamodel branch September 8, 2025 19:51
hugovk pushed a commit that referenced this pull request Sep 9, 2025
@miss-islington @AA-Turner
[3.14] gh-101100: Resolve reference warnings in reference/ (GH-138418) (
dee3428
#138680)

gh-101100: Resolve reference warnings in reference/ (GH-138418)
(cherry picked from commit 6831634)

Co-authored-by: Adam Turner <[email protected]>
lkollar pushed a commit to lkollar/cpython that referenced this pull request Sep 9, 2025
@AA-Turner @lkollar
pythongh-101100: Resolve reference warnings in reference/ (python#138418
db40d60
)
pablogsal pushed a commit that referenced this pull request Feb 15, 2026
@miss-islington @AA-Turner @StanFromIreland
[3.13] gh-101100: Resolve reference warnings in reference/ (GH-138418) (
e1dbe22
#138681)

Co-authored-by: Adam Turner <[email protected]>
Co-authored-by: Stan Ulbrych <[email protected]>
@encukou encukou mentioned this pull request May 13, 2026
Link to existing rules in compound_stmts.rst #149811
Merged
encukou added a commit that referenced this pull request May 14, 2026
@encukou
Link to existing rules in compound_stmts.rst (GH-149811)
c375292
In gh-138418, `!` was added to links to rules that don't exist in
the docs, in order to silence broken link warnings.
However, productionlist doesn't parse the `!`, which ends up in
the rendered documentation. (It's possible that gh-127835 broke
the `!` support.)

Replace the names with ones that appear in docs:

- `star_named_expression` in the grammar corresponds to
  `flexible_expression` in the docs
- `star_named_expressions` in the grammar corresponds to
  `flexible_expression_list` in the docs
- `named_expression` in the grammar corresponds to
  `assignment_expression` in the docs

Having two sets of names isn't great of course. Consolidating them
is tracked in (subissues of) gh-127833.
This was referenced May 14, 2026
[3.15] Link to existing rules in compound_stmts.rst (GH-149811) #149836
Merged
[3.14] Link to existing rules in compound_stmts.rst (GH-149811) #149837
Merged
[3.13] Link to existing rules in compound_stmts.rst (GH-149811) #149838
Merged
encukou added a commit that referenced this pull request May 14, 2026
@miss-islington @encukou
[3.15] Link to existing rules in compound_stmts.rst (GH-149811) (GH-1…
451f06b
…49836)

Link to existing rules in compound_stmts.rst (GH-149811)

In gh-138418, `!` was added to links to rules that don't exist in
the docs, in order to silence broken link warnings.
However, productionlist doesn't parse the `!`, which ends up in
the rendered documentation. (It's possible that gh-127835 broke
the `!` support.)

Replace the names with ones that appear in docs:

- `star_named_expression` in the grammar corresponds to
  `flexible_expression` in the docs
- `star_named_expressions` in the grammar corresponds to
  `flexible_expression_list` in the docs
- `named_expression` in the grammar corresponds to
  `assignment_expression` in the docs

Having two sets of names isn't great of course. Consolidating them
is tracked in (subissues of) gh-127833.
(cherry picked from commit c375292)

Co-authored-by: Petr Viktorin <[email protected]>
encukou added a commit that referenced this pull request May 14, 2026
@miss-islington @encukou
[3.14] Link to existing rules in compound_stmts.rst (GH-149811) (GH-1…
9a1f30f
…49837)

Link to existing rules in compound_stmts.rst (GH-149811)

In gh-138418, `!` was added to links to rules that don't exist in
the docs, in order to silence broken link warnings.
However, productionlist doesn't parse the `!`, which ends up in
the rendered documentation. (It's possible that gh-127835 broke
the `!` support.)

Replace the names with ones that appear in docs:

- `star_named_expression` in the grammar corresponds to
  `flexible_expression` in the docs
- `star_named_expressions` in the grammar corresponds to
  `flexible_expression_list` in the docs
- `named_expression` in the grammar corresponds to
  `assignment_expression` in the docs

Having two sets of names isn't great of course. Consolidating them
is tracked in (subissues of) gh-127833.
(cherry picked from commit c375292)

Co-authored-by: Petr Viktorin <[email protected]>
encukou added a commit that referenced this pull request May 14, 2026
@miss-islington @encukou
[3.13] Link to existing rules in compound_stmts.rst (GH-149811) (GH-1…
4da3e91
…49838)

Link to existing rules in compound_stmts.rst (GH-149811)

In gh-138418, `!` was added to links to rules that don't exist in
the docs, in order to silence broken link warnings.
However, productionlist doesn't parse the `!`, which ends up in
the rendered documentation. (It's possible that gh-127835 broke
the `!` support.)

Replace the names with ones that appear in docs:

- `star_named_expression` in the grammar corresponds to
  `flexible_expression` in the docs
- `star_named_expressions` in the grammar corresponds to
  `flexible_expression_list` in the docs
- `named_expression` in the grammar corresponds to
  `assignment_expression` in the docs

Having two sets of names isn't great of course. Consolidating them
is tracked in (subissues of) gh-127833.
(cherry picked from commit c375292)

Co-authored-by: Petr Viktorin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hugovk hugovk hugovk left review comments
@willingc willingc Awaiting requested review from willingc willingc is a code owner

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip news

Projects

Docs PRs
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@AA-Turner @hugovk
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.