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 + many PEPs: Add Sphinx Lint and fix errors #2816

Merged
merged 13 commits into from
Oct 6, 2022

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Oct 5, 2022

Lint PEPs using Sphinx Lint via pre-commit, already used for the devguide.

Let's run it with two checks disabled:

This PR also fixes the errors it finds.

Currently Sphinx Lint only runs on .rst files; I also fixed the errors it would find if the old .txt PEPs were renamed to .rst (which may happen as part of #10).

Whilst we're editing these PEPs:

  • Remove old Version and Last-Modified headers
  • Remove old emacs metadata
  • Redirect old BPO links to GH issues

@hugovk hugovk added bug infra Core infrastructure for building and rendering PEPs labels Oct 5, 2022
Copy link
Member

@AA-Turner AA-Turner left a comment

Choose a reason for hiding this comment

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

Looks OK

A

@JelleZijlstra JelleZijlstra merged commit 7fa80c3 into python:main Oct 6, 2022
@hugovk hugovk deleted the sphinx-lint branch October 6, 2022 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug infra Core infrastructure for building and rendering PEPs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants