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

[3.11] GH-95736: fix IsolatedAsyncioTestCase to initialize Runner bef… #96042

Merged
merged 1 commit into from
Aug 18, 2022

Conversation

kumaraditya303
Copy link
Contributor

@kumaraditya303 kumaraditya303 commented Aug 17, 2022

…ore calling setup functions (GH-95898)

(cherry picked from commit 9d51599)

Co-authored-by: Kumar Aditya [email protected]
Co-authored-by: Serhiy Storchaka [email protected]

…er before calling setup functions (pythonGH-95898)

(cherry picked from commit 9d51599)

Co-authored-by: Kumar Aditya <[email protected]>
Co-authored-by: Serhiy Storchaka <[email protected]>
@kumaraditya303 kumaraditya303 added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Aug 17, 2022
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @kumaraditya303 for commit ab95cf7 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Aug 17, 2022
Copy link
Member

@gvanrossum gvanrossum left a comment

Choose a reason for hiding this comment

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

I discussed this with @pablogsal and he says it's okay to backport. He will review and approve when he's seen the code.

@gvanrossum
Copy link
Member

Marked it as do-not-merge just to give @pablogsal a chance to review before it is merged. Pablo, once you agree that this can be merged into 3.11 please remove that label and approve the PR.

Copy link
Member

@pablogsal pablogsal left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks for pinging me! 👍

@pablogsal pablogsal merged commit b68ea2a into python:3.11 Aug 18, 2022
@kumaraditya303 kumaraditya303 deleted the backport-9d51599-3.11 branch August 18, 2022 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants