Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

tests: asyncio: collect garbage in run_until_complete_nesting#144168

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
benzea wants to merge1 commit intopython:main
base:main
Choose a base branch
Loading
frombenzea:main

Conversation

@benzea
Copy link

The test checks that coro1 emits a runtime warning when it is garbage collected. cpython will do this collection immediately, but other implementations such as PyPy might not. Add the appropriate call to support.gc_collect() to ensure the warning is emitted while still in the with statement.

The test checks that coro1 emits a runtime warning when it is garbagecollected. cpython will do this collection immediately, but otherimplementations such as PyPy might not. Add the appropriate call tosupport.gc_collect() to ensure the warning is emitted while still in thewith statement.
@python-cla-bot
Copy link

python-cla-botbot commentedJan 23, 2026
edited
Loading

All commit authors signed the Contributor License Agreement.

CLA signed

@bedevere-app
Copy link

Most changes to Pythonrequire a NEWS entry. Add one using theblurb_it web app or theblurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply theskip news label instead.

@bedevere-appbedevere-appbot added testsTests in the Lib/test dir awaiting review labelsJan 23, 2026
@picnixz
Copy link
Member

Is the only place? I would appreciate an issue for this as well.

@benzea
Copy link
Author

Is the only place? I would appreciate an issue for this as well.

It is the only place I noticed. But we only ran test_events.py against it for the pygobject GLibEventLoop integration tests. And even there we now switch to simply use the pypy sources instead, we pulled in the cpython version more by accident.

That said, I do think it is technically a bug, and there might be more … maybe it would actually be effective to diff the pypy version and see what they changed.

I suppose I can open a ticket and update the patch. I guess I'll wait a bit in case I find the motivation to check if more things need to be changed.

@picnixz
Copy link
Member

Still, please open a ticket with the specific bug. If there is more to do we will update the ticket. But this is something that needs to be tracked, at least for posterity.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@1st11st1Awaiting requested review from 1st11st1 is a code owner

@asvetlovasvetlovAwaiting requested review from asvetlovasvetlov is a code owner

@kumaraditya303kumaraditya303Awaiting requested review from kumaraditya303kumaraditya303 is a code owner

@willingcwillingcAwaiting requested review from willingcwillingc is a code owner

Assignees

No one assigned

Labels

awaiting reviewtestsTests in the Lib/test dir

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@benzea@picnixz

[8]ページ先頭

©2009-2026 Movatter.jp