use page titles as link text to avoid warnings#1532
Conversation
Previously we had scary red messages: ``` devguide/core-developers/memorialization.rst:5: WARNING: Duplicate explicit target name: "here". [docutils] devguide/core-developers/memorialization.rst:5: WARNING: Duplicate explicit target name: "here". [docutils] ```
|
Though TBH, that warning seems kind of dumb to me... Maybe we can disable it? /cc @hugovk |
|
A double trailing underscore make the link an "anonymous" one and avoids warnings: -`here <https://docs.github.com/en/site-policy/other-site-policies/github-deceased-user-policy>`_.
+`here <https://docs.github.com/en/site-policy/other-site-policies/github-deceased-user-policy>`__. |
|
sigh there's always some other twist to learn about .rst.... I've used double-underscore in its other form: Look over `there`__.
__ https://overthere.com |
ezio-melotti
left a comment
There was a problem hiding this comment.
In general the text of the link shouldn't be a generic text like "here" or "link", but should reflect the content of the page it links to, so rephrasing those links would be preferred.
If possible, the link should be the actual title of the page and/or the name of the service (e.g. "See also the Discourse best practices for deceased community members"). Doing this will also avoid warnings.
core-developers/memorialization.rst
Outdated
| The general policy regarding deceased users on GitHub is described | ||
| `here <https://docs.github.com/en/site-policy/other-site-policies/github-deceased-user-policy>`_. | ||
| The general policy regarding deceased users on GitHub is described on their | ||
| `GitHub Deceased User Policy <https://docs.github.com/en/site-policy/other-site-policies/github-deceased-user-policy>`_ |
There was a problem hiding this comment.
| `GitHub Deceased User Policy <https://docs.github.com/en/site-policy/other-site-policies/github-deceased-user-policy>`_ | |
| `Deceased User Policy <https://docs.github.com/en/site-policy/other-site-policies/github-deceased-user-policy>`_ |
core-developers/memorialization.rst
Outdated
| The general policy regarding deceased users on GitHub is described on their | ||
| `GitHub Deceased User Policy <https://docs.github.com/en/site-policy/other-site-policies/github-deceased-user-policy>`_ |
There was a problem hiding this comment.
Based on what we discussed, it would be better to include "GitHub" in the link, but the current sentence repeats it twice. I would either remove the other one:
| The general policy regarding deceased users on GitHub is described on their | |
| `GitHub Deceased User Policy <https://docs.github.com/en/site-policy/other-site-policies/github-deceased-user-policy>`_ | |
| The general policy regarding deceased users is described on their | |
| `GitHub Deceased User Policy <https://docs.github.com/en/site-policy/other-site-policies/github-deceased-user-policy>`_ page. |
or simply:
| The general policy regarding deceased users on GitHub is described on their | |
| `GitHub Deceased User Policy <https://docs.github.com/en/site-policy/other-site-policies/github-deceased-user-policy>`_ | |
| See also the `GitHub Deceased User Policy | |
| <https://docs.github.com/en/site-policy/other-site-policies/github-deceased-user-policy>`_ page. |
|
I've tweaked the GitHub link title to remove the duplication. |
Previously we had scary red messages:
📚 Documentation preview 📚: https://cpython-devguide--1532.org.readthedocs.build/