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

bpo-34429: Noted TemporaryFile behavior on non-Posix/non-Cygwin systems #31547

Merged
merged 2 commits into from Feb 25, 2022

Conversation

slateny
Copy link
Contributor

@slateny slateny commented Feb 24, 2022

I applied the suggested wording, but is it be accurate to say TemporaryFile 'behaves exactly like' NamedTemporaryFile? Saying 'behaves exactly like' seems to imply that TemporaryFile is an imitation of sorts, while the implementation of TemporaryFile = NamedTemporaryFile would appear to make them completely identical

https://bugs.python.org/issue34429

@iritkatriel
Copy link
Member

I agree, how about ‘is an alias for’?

Co-authored-by: Irit Katriel <1055913+iritkatriel@users.noreply.github.com>
@iritkatriel iritkatriel added needs backport to 3.9 only security fixes needs backport to 3.10 only security fixes labels Feb 25, 2022
@iritkatriel iritkatriel merged commit 53ecf9e into python:main Feb 25, 2022
@miss-islington
Copy link
Contributor

Thanks @slateny for the PR, and @iritkatriel for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-31567 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed needs backport to 3.10 only security fixes needs backport to 3.9 only security fixes labels Feb 25, 2022
@bedevere-bot
Copy link

GH-31568 is a backport of this pull request to the 3.9 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 25, 2022
…ms (pythonGH-31547)

(cherry picked from commit 53ecf9e)

Co-authored-by: slateny <46876382+slateny@users.noreply.github.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 25, 2022
…ms (pythonGH-31547)

(cherry picked from commit 53ecf9e)

Co-authored-by: slateny <46876382+slateny@users.noreply.github.com>
miss-islington added a commit that referenced this pull request Feb 25, 2022
…ms (GH-31547)

(cherry picked from commit 53ecf9e)

Co-authored-by: slateny <46876382+slateny@users.noreply.github.com>
miss-islington added a commit that referenced this pull request Feb 25, 2022
…ms (GH-31547)

(cherry picked from commit 53ecf9e)

Co-authored-by: slateny <46876382+slateny@users.noreply.github.com>
@slateny slateny deleted the s/tempfile branch February 26, 2022 01:59
hello-adam pushed a commit to hello-adam/cpython that referenced this pull request Jun 2, 2022
…ms (pythonGH-31547)

(cherry picked from commit 53ecf9e)

Co-authored-by: slateny <46876382+slateny@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants