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

fix: Remove pytest-asyncio dependency and fix more resource warnings #3883

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from

Conversation

euri10
Copy link
Contributor

@euri10 euri10 commented Nov 29, 2024

Description

  • removed pytest-asyncio and using anyio
  • fixed a few unclosed resources
  • made some test async where necessary

WIP

@github-actions github-actions bot added area/dependencies This PR involves changes to the dependencies size: large pr/external Triage Required 🏥 This requires triage labels Nov 29, 2024
@github-actions github-actions bot added the area/docs This PR involves changes to the documentation label Nov 29, 2024
@cofin cofin changed the title Remove pytest-asyncio dependency and fix more resource warnings fix: Remove pytest-asyncio dependency and fix more resource warnings Nov 29, 2024
… on a diff param alternative test_cors_simple_response and test_openapi_swagger, all the time its a unclosed socket on redis port 6397
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependencies This PR involves changes to the dependencies area/docs This PR involves changes to the documentation pr/external pr/internal size: large Triage Required 🏥 This requires triage type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants