Skip to content

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.

@CAM-Gerlach CAM-Gerlach requested a review from methane March 18, 2022 20:27
@CAM-Gerlach CAM-Gerlach requested a review from a team as a code owner March 18, 2022 20:27
@CAM-Gerlach CAM-Gerlach self-assigned this Mar 18, 2022
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