Skip to content

Update to canonical PEP URLs#2123

Merged
ChrisLovering merged 2 commits into
python-discord:mainfrom
hugovk:update-peps-url
Apr 3, 2022
Merged

Update to canonical PEP URLs#2123
ChrisLovering merged 2 commits into
python-discord:mainfrom
hugovk:update-peps-url

Conversation

@hugovk

@hugovk hugovk commented Mar 31, 2022

Copy link
Copy Markdown
Contributor

With the recent implementation of PEP 676 the canonical URL for PEPs has changed from, for example:

https://www.python.org/dev/peps/pep-0008/

to:

https://peps.python.org/pep-0008/

Redirects are in place so the old links still work, but let's update to use the new canonical form.

See also:


Also the RSS feed:

PEPS_RSS_URL = "https://www.python.org/dev/peps/peps.rss/"

Redirects to https://peps.python.org/peps.rss/ with trailing slash which is a 404.

https://peps.python.org/peps.rss without trailing slash works, so let's use this.

(I've also filed python/peps#2493, because we should keep that old redirect working too.)

@hugovk hugovk requested review from Den4200 and jb3 as code owners March 31, 2022 09:44

@wookie184 wookie184 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR!

@wookie184 wookie184 added p: 3 - low Low Priority t: enhancement Changes or improvements to existing features s: needs review Author is waiting for someone to review and approve labels Mar 31, 2022

@MaskDuck MaskDuck left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice

Comment thread bot/exts/info/pep.py
@ChrisLovering ChrisLovering enabled auto-merge April 3, 2022 18:07
@ChrisLovering ChrisLovering merged commit c02b158 into python-discord:main Apr 3, 2022
@hugovk hugovk deleted the update-peps-url branch April 3, 2022 18:30
@Xithrius Xithrius removed the s: needs review Author is waiting for someone to review and approve label May 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

p: 3 - low Low Priority t: enhancement Changes or improvements to existing features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants