-
-
Notifications
You must be signed in to change notification settings - Fork 142
Use AnyIO instead of pytest-asyncio for running async tests #242
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
Conversation
CodSpeed Performance ReportMerging #242 will not alter performanceComparing Summary
|
The linting failure is not due to my changes, it exists in |
Thanks for the contribution, @agronholm. Will review later. Never mind any linting issues, will fix them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thank you.
Will remove pytest-asyncio from pyproject.toml after merging.
Ah, not removing it from |
I'm actually defining them, but not using them in tox. Might improve that, good idea. Wanted to improve the dependency management anyway, probably switching from poetry to uv. |
You have dependency groups but they're using the Poetry specific syntax. Recent releases of Poetry should support the standard syntax under the |
Thanks, made a note to change that. But as already mentioned, might completely move away from Poetry anyway. |
@agronholm The project is now standard dependency groups, and uv instead of poetry, resulting in less duplication and more speed. Thanks for pushing me. |
This is the first step to implement #166.