
This issue trackerhas been migrated toGitHub, and is currentlyread-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.
Created on2018-10-22 08:11 bymatrixise, last changed2022-04-11 14:59 byadmin. This issue is nowclosed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 10036 | merged | matrixise,2018-10-22 09:47 | |
| Messages (11) | |||
|---|---|---|---|
| msg328240 -(view) | Author: Stéphane Wirtel (matrixise)*![]() | Date: 2018-10-22 08:11 | |
In the doc, we write PEP XYZ but there is a sphinx role for that, just replace PEP XYZ by :pep:`XYZ` and convert the :PEP: by :pep: | |||
| msg328246 -(view) | Author: Srinivas Reddy Thatiparthy(శ్రీనివాస్ రెడ్డి తాటిపర్తి) (thatiparthy)* | Date: 2018-10-22 11:38 | |
This gives us the hyperlink everywhere a PEP is referenced? | |||
| msg328247 -(view) | Author: Stéphane Wirtel (matrixise)*![]() | Date: 2018-10-22 11:47 | |
sure, it's the default behavior of this role, I have a PR with the fix, but Github has a big issue with the PRs :/ | |||
| msg328251 -(view) | Author: Serhiy Storchaka (serhiy.storchaka)*![]() | Date: 2018-10-22 12:47 | |
What is wrong with the current code? Why replace :PEP: with :pep:? | |||
| msg328252 -(view) | Author: Stéphane Wirtel (matrixise)*![]() | Date: 2018-10-22 12:54 | |
in fact, in the doc of Sphinx, it's just :pep: and not :PEP:seehttp://www.sphinx-doc.org/en/master/usage/restructuredtext/roles.html?highlight=%3Apep%3A#role-pepin this case, I think it's better to use the "reference" and use pep in lowercase. | |||
| msg328256 -(view) | Author: Srinivas Reddy Thatiparthy(శ్రీనివాస్ రెడ్డి తాటిపర్తి) (thatiparthy)* | Date: 2018-10-22 14:46 | |
I am -0 on this. I want to hear a more compelling argument than wanting to leverage an existing feature.IMHO, when i am reading the documentation i never bothered to go into the PEP docs - most of the times. I explicitly visited a PEP when i want to understand the rationale of a feature.My 2 cents. | |||
| msg328257 -(view) | Author: Stéphane Wirtel (matrixise)*![]() | Date: 2018-10-22 14:53 | |
@thatiparthyWhen you start to contribute to Python, and you see there is a PEP in the doc.1. What's a PEP?2. Where can I find the PEP XYZ?3. Am I on the right website for the PEP?if you don't know the concept of the PEPs, you will check with your favorite search engine because you are a developer. This is not the case for everybody.For a newcomer/beginner, read the right information just with a click, it's just the founding principles of the Web.create a link is not a real problem for a computer and for the end-user will be happy. | |||
| msg328258 -(view) | Author: Julien Palard (mdk)*![]() | Date: 2018-10-22 14:57 | |
When mentionning a PEP I see no reason not to point to it, and as it's easy to do with the pep role, I'm +1 on this. | |||
| msg328266 -(view) | Author: Karthikeyan Singaravelan (xtreak)*![]() | Date: 2018-10-22 17:34 | |
If I am understanding this correctly the :pep: role is already present and only that PEP references that don't have this role are being updated now?➜ cpython git:(master) rg ':pep:`\d+`' | grep 'rst' | grep -Ev 'NEWS|whatsnew' | wc 324 2686 27228References that don't have the role are being updated along with :PEP: changed to :pep: in the PR? (Around 30 entries) ➜ cpython git:(master) rg 'PEP \d+' | grep 'rst' | grep -Ev 'NEWS|whatsnew' | wc 21 228 1891➜ cpython git:(master) rg ':PEP:`\d+`' | grep 'rst' | grep -Ev 'NEWS|whatsnew' | wc 6 51 517I think older NEWS entries can be left behind like the PR changes whatsnew for Python 2.0, 2.1, 2.5 and they won't be back ported I personally feel little value in changing them though doc changes have been done on those files in 2016 and up to the reviewer.Thanks | |||
| msg328605 -(view) | Author: Éric Araujo (eric.araujo)*![]() | Date: 2018-10-26 21:01 | |
Docutils itself parses `RFC nnnn` and `PEP nnn` to replace with links.In some parts of the docs with many references, some devs (I think Raymond Hettinger for example) use tricks like `PEP\ nnn` to avoid getting twenty links in a row.So I don’t see what the replacement of `PEP nnn` with `:pep:`nnn`` adds. | |||
| msg328614 -(view) | Author: Brett Cannon (brett.cannon)*![]() | Date: 2018-10-26 22:58 | |
New changeset12e696b4f071ffe0d585b7f0d0d8020fd328bfdd by Brett Cannon (Stéphane Wirtel) in branch 'master':bpo-35042: Use the :pep: role where a PEP is specified (#10036)https://github.com/python/cpython/commit/12e696b4f071ffe0d585b7f0d0d8020fd328bfdd | |||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:59:07 | admin | set | github: 79223 |
| 2018-10-28 07:46:04 | mdk | set | status: open -> closed resolution: fixed stage: patch review -> resolved |
| 2018-10-26 22:58:35 | brett.cannon | set | nosy: +brett.cannon messages: +msg328614 |
| 2018-10-26 21:01:52 | eric.araujo | set | nosy: +eric.araujo messages: +msg328605 |
| 2018-10-22 17:34:00 | xtreak | set | nosy: +xtreak messages: +msg328266 |
| 2018-10-22 14:57:58 | mdk | set | nosy: +mdk messages: +msg328258 |
| 2018-10-22 14:53:58 | matrixise | set | messages: +msg328257 |
| 2018-10-22 14:46:14 | thatiparthy | set | messages: +msg328256 |
| 2018-10-22 12:54:19 | matrixise | set | messages: +msg328252 |
| 2018-10-22 12:47:52 | serhiy.storchaka | set | nosy: +serhiy.storchaka messages: +msg328251 |
| 2018-10-22 11:47:27 | matrixise | set | messages: +msg328247 |
| 2018-10-22 11:38:00 | thatiparthy | set | nosy: +thatiparthy messages: +msg328246 |
| 2018-10-22 09:47:52 | matrixise | set | keywords: +patch stage: patch review pull_requests: +pull_request9374 |
| 2018-10-22 08:11:03 | matrixise | create | |