Skip to content

Use pep role instead of url #121611

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Sep 17, 2024
Merged

Use pep role instead of url #121611

merged 2 commits into from
Sep 17, 2024

Conversation

rffontenelle
Copy link
Contributor

@rffontenelle rffontenelle commented Jul 11, 2024

@bedevere-app bedevere-app bot added docs Documentation in the Doc dir skip news awaiting review labels Jul 11, 2024
@hugovk
Copy link
Member

hugovk commented Sep 13, 2024

There's some others in here that could be updated too, would you like to include some more in this PR?

rg peps.python.org Doc
Doc/tools/templates/indexsidebar.html
13:  <li><a href="https://peps.python.org/">{% trans %}PEP Index{% endtrans %}</a></li>

Doc/glossary.rst
1175:      <https://peps.python.org/pep-0387/#soft-deprecation>`_.

Doc/whatsnew/3.4.rst
223:__ https://peps.python.org/pep-0453/#recommendations-for-downstream-distributors

Doc/whatsnew/3.7.rst
356:`Measurements <https://peps.python.org/pep-0564/#annex-clocks-resolution-in-python>`_

Doc/whatsnew/3.12.rst
157:  See `the migration guide <https://peps.python.org/pep-0632/#migration-advice>`_

Doc/whatsnew/2.7.rst
2688:__ https://peps.python.org/pep-0477/#disabling-ensurepip-by-downstream-distributors

Doc/whatsnew/2.0.rst
133:https://peps.python.org/.  As of September 2000, there are 25 PEPs, ranging

Doc/faq/general.rst
315:maintained anymore <https://peps.python.org/pep-0373/>`_.
350:See https://peps.python.org/ for the Python Enhancement Proposals

Doc/c-api/refcounting.rst
65:   For at least `some objects <https://peps.python.org/pep-0683/>`_,
133:   For at least `some objects <https://peps.python.org/pep-0683/>`_,

Doc/c-api/code.rst
100:    <https://peps.python.org/pep-0626/#out-of-process-debuggers-and-profilers>`_.

Doc/library/sqlite3.rst
542:   .. _threadsafety: https://peps.python.org/pep-0249/#threadsafety

Doc/library/functools.rst
238:            resource = f'https://peps.python.org/pep-{num:04d}'

Doc/library/ast.rst
2037:     <https://peps.python.org/pep-3115/>`_.

Doc/library/wsgiref.rst
787:   <https://peps.python.org/pep-3333/#the-start-response-callable>`_
801:   <https://peps.python.org/pep-3333/#input-and-error-streams>`_.
806:   <https://peps.python.org/pep-3333/#input-and-error-streams>`_.
811:   <https://peps.python.org/pep-3333/#optional-platform-specific-file-handling>`_.

@rffontenelle
Copy link
Contributor Author

rffontenelle commented Sep 17, 2024

@hugovk fixed the occurrences of pep URL you reported, except for the ones below because they didn't seem to be the case of using the pep role. And Doc/library/ast.rst was fixed already before me.

Doc/tools/templates/indexsidebar.html
13:  <li><a href="https://peps.python.org/">{% trans %}PEP Index{% endtrans %}</a></li>

Doc/whatsnew/2.0.rst
133:https://peps.python.org/.  As of September 2000, there are 25 PEPs, ranging

Doc/faq/general.rst
350:See https://peps.python.org/ for the Python Enhancement Proposals

Doc/library/functools.rst
238:            resource = f'https://peps.python.org/pep-{num:04d}'

NOTE: force-pushed below to solve a merge conflict in glossary.rst

@rffontenelle rffontenelle changed the title Use pep role instead of url in ast.rst Use pep role instead of url Sep 17, 2024
@hugovk
Copy link
Member

hugovk commented Sep 17, 2024

Thank you!

@hugovk hugovk merged commit 33eeccf into python:main Sep 17, 2024
25 checks passed
@miss-islington-app
Copy link

Thanks @rffontenelle for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 17, 2024
(cherry picked from commit 33eeccf)

Co-authored-by: Rafael Fontenelle <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Sep 17, 2024

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 17, 2024
(cherry picked from commit 33eeccf)

Co-authored-by: Rafael Fontenelle <[email protected]>
@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Sep 17, 2024
@bedevere-app
Copy link

bedevere-app bot commented Sep 17, 2024

GH-124173 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Sep 17, 2024
hugovk pushed a commit that referenced this pull request Sep 17, 2024
@rffontenelle rffontenelle deleted the patch-6 branch September 20, 2024 15:42
savannahostrowski pushed a commit to savannahostrowski/cpython that referenced this pull request Sep 22, 2024
Yhg1s pushed a commit that referenced this pull request Sep 23, 2024
Use pep role instead of url (GH-121611)
(cherry picked from commit 33eeccf)

Co-authored-by: Rafael Fontenelle <[email protected]>
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
None yet
Development

Successfully merging this pull request may close these issues.

2 participants