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

Infra: gitignore generated /topic and wipe via make clean #2683

Merged
merged 1 commit into from
Jul 10, 2022

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Jun 28, 2022

Fixes #2659.

@hugovk hugovk added the infra Core infrastructure for building and rendering PEPs label Jun 28, 2022
@hugovk hugovk requested a review from pradyunsg June 28, 2022 08:36
Copy link
Member

@CAM-Gerlach CAM-Gerlach left a comment

Choose a reason for hiding this comment

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

For the record, IMO the general approach @pradyunsg proposed, checking in actual topic index pages to the source and using a custom pep-table directive to generate all the tabs would be a more sustainable long-term solution than adding our own custom reST source file generation and somewhat hackily getting Sphinx to recognize them, but may as well do this for now.

@CAM-Gerlach CAM-Gerlach merged commit bf131f0 into python:main Jul 10, 2022
@hugovk hugovk deleted the gitignore-topic branch July 11, 2022 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra Core infrastructure for building and rendering PEPs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Running build.py to render PEPs leaves residual files in a new topic/ folder
2 participants