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

PEP 686: Update URLs, PEP references, Discussions-To and CODEOWNERS per PEP 12 #2444

Merged
merged 3 commits into from
Mar 19, 2022

Conversation

CAM-Gerlach
Copy link
Member

@CAM-Gerlach CAM-Gerlach commented Mar 18, 2022

Technical-only updates to PEP 686 (PEP-0686) per PEP 12 (PEP-0012):

  • Use the :pep: role instead of hardcoded PEP URLs (the checkers are supposed to catch this, but I was still in the process of updating it to check for the new URL format; I'll have that very shortly)
  • Use links instead of footnotes for references (see Discourage using References? #2130)
  • Add the Discussions-To and Post-History headers now that the thread is live (it should be updated whenever a new discussion thread for the PEP is posted, so people know where to go to give feedback)
  • Add @methane as the owner of PEP 686 in CODEOWNERS

@methane in the process, I also noticed some grammar issues in the PEP, but I refrained from changing the text here. If you'd like me to follow up with another PR for your review addressing them, to help make the PEP easier for others to read and understand clearly, I would be happy to. Just let me know; it is up to you as the author of the PEP.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants