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 706: Add some notes, remove open issue, fix mistakes & formatting #3023

Merged
merged 3 commits into from
Feb 22, 2023

Conversation

encukou
Copy link
Member

@encukou encukou commented Feb 22, 2023

  • Add a section on adding filters to zipfile. Not quite a “rejected idea”, but IMO the PEP is good enough without it.
  • Add a note on a registration mechanism
  • Remove Open Issue “How far should this be backported?”. That'll be up to release managers, no need to put it in the PEP.
  • Fix a few obvious mistakes, typos, formatting

  • Change is either:
    • To a Draft PEP
    • To an Accepted or Final PEP, with Steering Council approval
    • To fix an editorial issue (markup, typo, link, header, etc)
  • PR title prefixed with PEP number (e.g. PEP 123: Summary of changes)

- Remove Open Issue “How far should this be backported?”.
  That'll be up to release managers, no need to put it in the PEP.
- Add a section on adding filters to zipfile. Not *quite* a
  “rejected idea”, but IMO the PEP is good enough without it.
- Add a note on a registration mechanism
- Fix a few obvious mistakes, typos, formatting
pep-0706.rst Show resolved Hide resolved
pep-0706.rst Outdated Show resolved Hide resolved
Co-authored-by: Jelle Zijlstra <[email protected]>
@encukou
Copy link
Member Author

encukou commented Feb 22, 2023

Thanks for the review!

@encukou encukou merged commit 706b7aa into python:main Feb 22, 2023
@encukou encukou deleted the tarfile-dir-traversal branch February 22, 2023 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants