Skip to content

Conversation

@humbdrag
Copy link
Contributor

@humbdrag humbdrag commented Jan 24, 2022

Finishing up https://bugs.python.org/issue29890, original PR #816.
By adding the same tests, it seems that the issue is already addressed in the latest version.

#74076

https://bugs.python.org/issue29890

@humbdrag
Copy link
Contributor Author

@Kentzo sorry for tagging but any thoughts?

@Kentzo
Copy link
Contributor

Kentzo commented Mar 30, 2022

Totally makes sense. But I'm not pulling the strings here :)

@JelleZijlstra JelleZijlstra self-assigned this May 2, 2022
@JelleZijlstra
Copy link
Member

I'll backport to all versions where the tests pass.

@JelleZijlstra JelleZijlstra merged commit b295a92 into python:main May 3, 2022
@miss-islington
Copy link
Contributor

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

@bedevere-bot
Copy link

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

@bedevere-bot
Copy link

GH-92243 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 May 3, 2022
…nGH-30862)

Co-authored-by: Jelle Zijlstra <[email protected]>
(cherry picked from commit b295a92)

Co-authored-by: Humbled Drugman <[email protected]>
miss-islington added a commit that referenced this pull request May 3, 2022
)

Co-authored-by: Jelle Zijlstra <[email protected]>
(cherry picked from commit b295a92)

Co-authored-by: Humbled Drugman <[email protected]>
miss-islington added a commit that referenced this pull request May 3, 2022
)

Co-authored-by: Jelle Zijlstra <[email protected]>
(cherry picked from commit b295a92)

Co-authored-by: Humbled Drugman <[email protected]>
hello-adam pushed a commit to hello-adam/cpython that referenced this pull request Jun 2, 2022
…nGH-30862)

Co-authored-by: Jelle Zijlstra <[email protected]>
(cherry picked from commit b295a92)

Co-authored-by: Humbled Drugman <[email protected]>
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.

6 participants