[PEP 817] Switch To Informational Track - #80
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates PEP 817 to the Informational track and correspondingly trims/restructures the document, including removing the standalone variant-metadata JSON schema appendix and schema file.
Changes:
- Switches the PEP header from Standards Track to Informational and adds an “Informational PEP” section near the top.
- Removes the appendix that literal-included the variant metadata JSON schema and deletes the standalone
variant_schema.json. - Restructures several sections in
pep-0817.rst, including substantial removals of previously specified material.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 8 comments.
| File | Description |
|---|---|
peps/pep-0817.rst |
Updates PEP type to Informational and significantly restructures/removes content, adding new framing sections. |
peps/pep-0817/appendix-variant-metadata-json-schema.rst |
Removes the appendix that embedded the JSON schema via literalinclude. |
peps/pep-0817/variant_schema.json |
Deletes the standalone JSON Schema file that was referenced by the removed appendix. |
Suppressed comments (2)
peps/pep-0817.rst:951
- Unresolved “” placeholder text is checked in right before the example. Replace it with a brief lead-in sentence (and avoid the inline “FLIP it on its head” note).
<TODO> FLIP it on its head => information hardcoded in the pyproject.toml
peps/pep-0817.rst:1366
- Unresolved “ Rephrase” placeholder should be replaced with final prose (or removed) before merging.
<TODO> Rephrase
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| This PEP presents the minimal scope required to meet modern heterogenous | ||
| system needs. It leaves aspects beyond the minimal scope to evolve via | ||
| tools or future PEPs. A non-exhaustive list of these aspects include: |
Documentation build overview
6 files changed ·
|
f8fffdd to
97fc8a2
Compare
|
Thanks @DEKHTIARJonathan. Looking at the current state of this PR, it's a good step towards what we need. Given that the diff is already quite large, I'd like to propose keeping the review here fairly minimal, merging this PR as a "remove Specification, rework the structure" and then work on two follow-up PRs:
WDYT? |
|
Do I understand correctly that the purpose of PEP 817 henceforth will be to define a minimal "end-to-end" view of what PEP 825 and followup PEPs will design in detail? |
|
I'd say that that's only one part. The Specification should be removed completely (it's Informational), so the outline is something like:
I think with minimal "end-to-end" view you're referring to the last bullet only. |
|
@mgorny what we discussed with Ralf @ EuroPython was to capture the context and reasoning behind the variant design and the intended final goal we are collectively aiming at. |
Yeah, that's the part I was referring to. |
|
Let's merge it as-is and go from there, I guess? |
|
Agreed, merged. How about you work on the last bullet and I work on the third one @mgorny? |
|
Sure. Do you see it as replacing the current "overview" or separate? |
|
I think yes, start at
Then also give it a fresh look - what, if anything, is completely missing (static file for overrides with use cases that we accepted in the meantime for example?) and see if the whole design explanation is coherent. |
No description provided.