Skip to content
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

Update to canonical PEP URLs #2123

Merged
merged 2 commits into from Apr 3, 2022

Conversation

hugovk
Copy link
Contributor

@hugovk hugovk commented Mar 31, 2022

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
Copy link
Contributor

@wookie184 wookie184 left a comment

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
Copy link
Contributor

@MaskDuck MaskDuck left a comment

Choose a reason for hiding this comment

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

nice

@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.

None yet

6 participants