Skip to content

Include the PEP index in peps.json - #2567

Merged
AA-Turner merged 1 commit into
python:mainfrom
wookie184:pep-0-in-peps-json
May 1, 2022
Merged

Include the PEP index in peps.json#2567
AA-Turner merged 1 commit into
python:mainfrom
wookie184:pep-0-in-peps-json

Conversation

@wookie184

@wookie184wookie184 commented May 1, 2022

Copy link
Copy Markdown
Contributor

pep 0 is a bit different from other peps given that it's autogenerated, but I still think it makes sense to include it in the api/peps.json endpoint.

Page in build preview: https://pep-previews--2567.org.readthedocs.build/api/peps.json

@wookie184
wookie184 requested a review from AA-Turner as a code ownerMay 1, 2022 12:46
@AA-TurnerAA-Turner changed the title Include pep 0 in api/peps.jsonInclude the PEP index in peps.jsonMay 1, 2022
@AA-Turner
AA-Turner merged commit 278070a into python:mainMay 1, 2022
@AA-Turner

Copy link
Copy Markdown
Member

Thanks, PEP 0 is still a PEP so should be included (although fair enough to exclude PEP 0 from itself!)

A

@CAM-GerlachCAM-Gerlach added the infra Core infrastructure for building and rendering PEPs label May 3, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infraCore infrastructure for building and rendering PEPs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@wookie184@AA-Turner@CAM-Gerlach