Skip to content

gh-127949: fix DeprecationWarning in test_inspect.py - #128215

Merged
kumaraditya303 merged 3 commits into
python:mainfrom
graingert:fix-test-inspect-warnings
Dec 24, 2024
Merged

gh-127949: fix DeprecationWarning in test_inspect.py#128215
kumaraditya303 merged 3 commits into
python:mainfrom
graingert:fix-test-inspect-warnings

Conversation

@graingert

@graingertgraingert commented Dec 24, 2024•
edited
Loading

Copy link
Copy Markdown
Contributor

This works by avoiding asyncio, which is overkill just for running a noop coroutine

@bedevere-appbedevere-appBot added the tests Tests in the Lib/test dir label Dec 24, 2024
@graingertgraingert changed the title fix DeprecationWarning in test_inspect.pygh-127949: fix DeprecationWarning in test_inspect.pyDec 24, 2024
@bedevere-appbedevere-appBot mentioned this pull request Dec 24, 2024
Closed
7 tasks
graingert
graingert commented Dec 24, 2024
View reviewed changes
Comment threadLib/test/test_inspect/test_inspect.py Outdated
@graingert
graingertforce-pushed the fix-test-inspect-warnings branch from 491d488 to 1b13706CompareDecember 24, 2024 08:50
@graingert
graingert marked this pull request as ready for review December 24, 2024 08:58
kumaraditya303
kumaraditya303 reviewed Dec 24, 2024
View reviewed changes
Comment threadLib/test/test_inspect/test_inspect.py
kumaraditya303
kumaraditya303 approved these changes Dec 24, 2024
View reviewed changes

@kumaraditya303kumaraditya303 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@kumaraditya303
kumaraditya303 merged commit 3f6a618 into python:mainDec 24, 2024
@graingert
graingert deleted the fix-test-inspect-warnings branch December 24, 2024 09:43
srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Jan 8, 2025
@graingert@srinivasreddy
pythongh-127949: fixDeprecationWarningin test_inspect.py (python#…
5da4345
…128215)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kumaraditya303kumaraditya303kumaraditya303 approved these changes

Assignees

No one assigned

Labels

skip newstestsTests in the Lib/test dir

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@graingert@kumaraditya303