| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
…classes ParamSpecArgs and ParamSpecKwargs are classes (the runtime types of the args and kwargs attributes of a ParamSpec), but were documented with the ".. data::" directive. The existing ":class:`ParamSpecArgs`" and ":class:`ParamSpecKwargs`" cross-references in the typing docs and the 3.10 What's New page therefore fail to resolve against a py:class target. Switch the stacked entry to ".. class::" so the references resolve.
Documentation build overview1 file changed ± library/typing.html |
Sorry, something went wrong.
|
Backward-compatibility check. The only downstream :data: reference found is typing_extensions (doc/index.rst), which is maintained alongside CPython. :class: references to ParamSpecArgs already exist in typing.rst and whatsnew/3.10.rst. Low risk. |
Sorry, something went wrong.
There was a problem hiding this comment.
LGTM. 👍
Sorry, something went wrong.
|
Thanks @gaborbernat for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14, 3.15. |
Sorry, something went wrong.
|
GH-150763 is a backport of this pull request to the 3.15 branch. |
Sorry, something went wrong.
|
GH-150764 is a backport of this pull request to the 3.14 branch. |
Sorry, something went wrong.
|
GH-150765 is a backport of this pull request to the 3.13 branch. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
typing.ParamSpecArgs and typing.ParamSpecKwargs are classes (the runtime types of the args and kwargs attributes of a ParamSpec), but the documentation marks them with the .. data:: directive. The existing :class: cross-references to them in Doc/library/typing.rst and Doc/whatsnew/3.10.rst therefore resolve against a missing py:class target.
Switch the stacked entry to .. class:: so the references resolve. Both names share a single entry, so both are covered.
Refs: gh-89554. Documentation-only change, so no Misc/NEWS entry (skip news).