PEP 764: Second round of updates#4386
PEP 764: Second round of updates#4386JelleZijlstra merged 3 commits intopython:mainpython/peps:mainfrom
Conversation
peps/pep-0764.rst
Outdated
| def test_values() -> {"int": int, "str": str}: | ||
| return {"int": 42, "str": "test"} | ||
|
|
||
| Support for this PEP is added in `this pull request <https://github.com/python/mypy/pull/18889>`_. |
There was a problem hiding this comment.
To fix the build, add a second underscore to make an "anonymous" reference, so it's not duplicated:
| Support for this PEP is added in `this pull request <https://github.com/python/mypy/pull/18889>`_. | |
| Support for this PEP is added in `this pull request <https://github.com/python/mypy/pull/18889>`__. |
Plus we could use more accessible link text instead:
https://www.a11yproject.com/posts/creating-valid-and-accessible-links/#write-helpful-link-text
There was a problem hiding this comment.
Do you have suggestions to make it more accessible? The way I understand what's described in your link is that the link text should describe what it points to, and this pull request seems to describe the target correctly? (Perhaps mypy should be mentioned in the text).
There was a problem hiding this comment.
If someone is navigating just via links, they'll hear "this pull request", "this pull request" for both.
Perhaps we could mention the repo names? For example, "this mypy pull request", "this typing_extensions pull request".
|
@erictraut are you OK with merging this? |
|
Yes, looks good to me. |
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
To avoid yet another update, might be best to wait for the outcome of PEP 728 so that I can update this PEP with respect to the closed behavior.
cc @erictraut
PEP 123: Summary of changes)