Skip to content

Conversation

sobolevn
Copy link
Member

@sobolevn sobolevn commented Oct 26, 2022

Copy link
Member

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

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

Should we maybe add a short news entry? The test made it into 3.11.0.

Otherwise this looks good! You beat the PR I was writing to the finish line ;)

@sobolevn
Copy link
Member Author

I've never seen a NEWS entry for tests. If others say so - I surely will add it.

@AlexWaygood
Copy link
Member

We usually skip news for PRs that improve test coverage etc, but this is fixing a bug in the test suite that existed in a previous release of Python, so I'd argue for including a short news entry.

…2R.rst

Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
@miss-islington
Copy link
Contributor

@sobolevn: Status check is done, and it's a failure or timed out ❌.

@AlexWaygood
Copy link
Member

The test_asyncio failure is unrelated and will be fixed by #98704.

@AlexWaygood AlexWaygood merged commit 9495360 into python:main Oct 26, 2022
@miss-islington
Copy link
Contributor

Thanks @sobolevn for the PR, and @AlexWaygood for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-98717 is a backport of this pull request to the 3.11 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label Oct 26, 2022
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 26, 2022
…thonGH-98714)

Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
(cherry picked from commit 9495360)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
miss-islington added a commit that referenced this pull request Oct 26, 2022
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
(cherry picked from commit 9495360)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants