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-45401: Fix a resource warning in test_logging #28864

Merged

Conversation

serhiy-storchaka
Copy link
Member

@serhiy-storchaka serhiy-storchaka commented Oct 11, 2021

@vstinner
Copy link
Member

Oh, you created a fix almost the same time than me :-D But you was faster, I close my fix: PR #28865.

Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM.

I plan to backport the isfile() change with this change to 3.10, and then backport the 3.10 change to 3.9.

@serhiy-storchaka
Copy link
Member Author

Only on a half of minute! 🏁

@serhiy-storchaka serhiy-storchaka merged commit 15188b1 into python:main Oct 11, 2021
@serhiy-storchaka serhiy-storchaka deleted the test_logging-close-fh branch October 11, 2021 08:54
@vstinner vstinner added needs backport to 3.9 only security fixes needs backport to 3.10 only security fixes labels Oct 11, 2021
@miss-islington
Copy link
Contributor

Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 3.10.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

@bedevere-bot
Copy link

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

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Oct 11, 2021
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 11, 2021
(cherry picked from commit 15188b1)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 11, 2021
(cherry picked from commit 15188b1)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label Oct 11, 2021
@bedevere-bot
Copy link

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

serhiy-storchaka added a commit that referenced this pull request Oct 12, 2021
(cherry picked from commit 15188b1)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
serhiy-storchaka added a commit that referenced this pull request Oct 12, 2021
(cherry picked from commit 15188b1)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants