Skip to content

Conversation

@graingert
Copy link
Contributor

@graingertgraingert commented Oct 12, 2023

@graingertgraingert changed the title allow setting the Runner(loop_factory=...) from IsolatedAsyncioTestCasegh-10774: allow setting the Runner(loop_factory=...) from IsolatedAsyncioTestCaseOct 12, 2023
@graingertgraingert changed the title gh-10774: allow setting the Runner(loop_factory=...) from IsolatedAsyncioTestCasegh-110774: allow setting the Runner(loop_factory=...) from IsolatedAsyncioTestCaseOct 12, 2023
@graingertgraingertforce-pushed the unittest-loop-factory branch from 47669b3 to 0fbb894CompareOctober 12, 2023 15:20
@graingertgraingert marked this pull request as ready for review October 12, 2023 15:32
Copy link
Contributor

@willingcwillingc left a comment

Choose a reason for hiding this comment

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

Thanks @graingert. This looks good to me.

Copy link
Member

@gvanrossumgvanrossum left a comment

Choose a reason for hiding this comment

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

Yup, thanks!

@gvanrossumgvanrossum merged commit 7705306 into python:mainOct 31, 2023
@graingertgraingert deleted the unittest-loop-factory branch October 31, 2023 22:32
FullteaR pushed a commit to FullteaR/cpython that referenced this pull request Nov 3, 2023
…atedAsyncioTestCase (python#110776) Co-authored-by: Łukasz Langa <lukasz@langa.pl> Co-authored-by: Carol Willing <carolcode@willingconsulting.com>
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
…atedAsyncioTestCase (python#110776) Co-authored-by: Łukasz Langa <lukasz@langa.pl> Co-authored-by: Carol Willing <carolcode@willingconsulting.com>
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
…atedAsyncioTestCase (python#110776) Co-authored-by: Łukasz Langa <lukasz@langa.pl> Co-authored-by: Carol Willing <carolcode@willingconsulting.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@graingert@willingc@gvanrossum@ambv