FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.15] Link to existing rules in compound_stmts.rst (GH-149811) by miss-islington · Pull Request #149836 · python/cpython · GitHub

/ cpython Public

[3.15] Link to existing rules in compound_stmts.rst (GH-149811) - #149836

Merged
encukou merged 1 commit into
python:3.15from
miss-islington:backport-c375292-3.15
May 14, 2026
Merged

[3.15] Link to existing rules in compound_stmts.rst (GH-149811)#149836
encukou merged 1 commit into
python:3.15from
miss-islington:backport-c375292-3.15

Conversation

Copy link
Copy Markdown
Contributor

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 encukou@gmail.com

In pythongh-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 pythongh-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) pythongh-127833.
(cherry picked from commit c375292)

Co-authored-by: Petr Viktorin <encukou@gmail.com>
bedevere-app Bot added docs Documentation in the Doc dir skip news labels May 14, 2026
github-project-automation Bot moved this to Todo in Docs PRs May 14, 2026
encukou merged commit 451f06b into python:3.15 May 14, 2026
40 checks passed
github-project-automation Bot moved this from Todo to Done in Docs PRs May 14, 2026
miss-islington deleted the backport-c375292-3.15 branch May 14, 2026 17:09
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip issue skip news

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL