Skip to content

gh-104638: make TarFile.extraction_filter future release references more clear #109908

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

Closed
wants to merge 1 commit into from

Conversation

hadrizi
Copy link

@hadrizi hadrizi commented Sep 26, 2023

Documentation for TarFile.extraction_filter future releases are referenced as "In python 3.14+ ..." which may be confusing thus plus symbol is removed.


📚 Documentation preview 📚: https://cpython-previews--109908.org.readthedocs.build/

@ghost
Copy link

ghost commented Sep 26, 2023

All commit authors signed the Contributor License Agreement.
CLA signed

@@ -557,7 +557,7 @@ be finalized; only the internally used file object will be closed. See the
and fall back to the :func:`fully_trusted <fully_trusted_filter>` filter,
whose dangerous behavior matches previous versions of Python.

In Python 3.14+, leaving ``extraction_filter=None`` will cause
In Python 3.14, leaving ``extraction_filter=None`` will cause
Copy link
Contributor

Choose a reason for hiding this comment

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

Isn't that a bit redundant?

Suggested change
In Python 3.14, leaving ``extraction_filter=None`` will cause
Leaving ``extraction_filter=None`` will cause

And can't we merge this with the previous paragraph?

@hugovk
Copy link
Member

hugovk commented Oct 9, 2024

Thanks for the PR, made redundant by #122002 which removed this bit of text.

@hugovk hugovk closed this Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review docs Documentation in the Doc dir skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants