Skip to content

PEP 665: link to "replaced by" #3874

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

Closed
davidszotten opened this issue Jul 26, 2024 · 7 comments · Fixed by #3875
Closed

PEP 665: link to "replaced by" #3874

davidszotten opened this issue Jul 26, 2024 · 7 comments · Fixed by #3875

Comments

@davidszotten
Copy link

Some peps replace other previous peps. it would be nice if the page for replaced peps also linked to their replacements

@hugovk hugovk added the infra Core infrastructure for building and rendering PEPs label Jul 26, 2024
@hugovk hugovk changed the title link to "replaced by" Infra: link to "replaced by" Jul 26, 2024
@hugovk
Copy link
Member

hugovk commented Jul 26, 2024

Please can you give an example?

@davidszotten
Copy link
Author

@davidszotten
Copy link
Author

(this was prompted by me trying to find the new one and accidentally initially landing on the old one)

@hugovk
Copy link
Member

hugovk commented Jul 26, 2024

We have a Superseded-By header for this.

For example, compare https://peps.python.org/pep-3153/ -> https://peps.python.org/pep-3156/

Let's also add Superseded-by: 751 to PEP 665: please see PR #3875.

@hugovk hugovk changed the title Infra: link to "replaced by" PEP 665: link to "replaced by" Jul 26, 2024
@hugovk hugovk removed the infra Core infrastructure for building and rendering PEPs label Jul 26, 2024
@davidszotten
Copy link
Author

🎉 thanks!

@davidszotten
Copy link
Author

davidszotten commented Jul 26, 2024

looks like there are a bunch more inconsistencies between the replaces and superseded-by (thanks peps api!)

should they all be in sync? do we care enough to fix? to add some tool to detect future ones?

e.g.

-    "3124": "245, 246",
-    "402": "382",
+    "387": "5",
-    "426": "345",
-    "443": "245, 246, 3124",
-    "449": "381",
-    "452": "247",
-    "464": "381",
-    "487": "422",
-    "600": "513, 571, 599",
+    "600": "599",
+    "621": "631",
-    "723": "722",
-    "742": "724",

@hugovk
Copy link
Member

hugovk commented Aug 6, 2024

@AA-Turner What do you think? ^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants