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.

Sign up for GitHub

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

Jump to bottom

Infra: don't wrap PEP numbers in PEP 0 tables #2524

Merged
merged 1 commit into from
Apr 13, 2022
Merged

Infra: don't wrap PEP numbers in PEP 0 tables #2524

merged 1 commit into from
Apr 13, 2022

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Apr 13, 2022 •
edited
Loading

Fixes #2520.

The four-digit PEP numbers were wrapping so the last digit is on a second line.

Before

image

https://peps.python.org/

After

image

https://pep-previews--2524.org.readthedocs.build/

@hugovk hugovk added the infra Core infrastructure for building and rendering PEPs label Apr 13, 2022
@hugovk hugovk requested a review from AA-Turner as a code owner April 13, 2022 06:40
@hugovk hugovk requested a review from CAM-Gerlach April 13, 2022 06:48
@hugovk hugovk mentioned this pull request Apr 13, 2022
Closed
@JelleZijlstra JelleZijlstra merged commit 642ad67 into python:main Apr 13, 2022
@JelleZijlstra
Copy link
Member

Thanks @hugovk !

@hugovk hugovk deleted the infra-pep0-table-whitespace branch April 13, 2022 18:27
@CAM-Gerlach
Copy link
Member

Oops, totally missed that—my fault! Thanks @hugovk

@erlend-aasland erlend-aasland mentioned this pull request May 13, 2022
Closed
@erlend-aasland erlend-aasland mentioned this pull request Jun 27, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AA-Turner AA-Turner Awaiting requested review from AA-Turner AA-Turner is a code owner

@CAM-Gerlach CAM-Gerlach Awaiting requested review from CAM-Gerlach

Assignees
No one assigned
Labels
infra Core infrastructure for building and rendering PEPs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

PEP zero formatting truncates 4-digit PEP numbers
3 participants
@hugovk @JelleZijlstra @CAM-Gerlach