Skip to content

Test a version of Trio that is compatible with AnyIO#3139

Merged
pquentin merged 1 commit intoelastic:mainfrom
pquentin:anyio-trio
Feb 5, 2026
Merged

Test a version of Trio that is compatible with AnyIO#3139
pquentin merged 1 commit intoelastic:mainfrom
pquentin:anyio-trio

Conversation

@pquentin
Copy link
Copy Markdown
Member

As suggested by @agronholm in #3089 (comment)

"python-dateutil",
"typing-extensions",
"sniffio",
"anyio",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Shouldn't we also add a lower bound for AnyIO? Some folks are still using 3.x from what I can see from the download charts.

@agronholm
Copy link
Copy Markdown

What also caught my eye was the explicit use of sniffio. Why is there a helper function _sleep() instead of just using anyio.sleep()? And the conftest.py use of sniffio could just be replaced by the use of the anyio_backend_name fixture. That would let you drop the explicit dependency on sniffio.

@pquentin
Copy link
Copy Markdown
Member Author

pquentin commented Oct 31, 2025

Good point! Initially anyio wasn't a runtime dependency, but then we needed a queue, and I did not clean up that part.

@agronholm
Copy link
Copy Markdown

Yup, makes perfect sense.

@pquentin pquentin merged commit 96706df into elastic:main Feb 5, 2026
16 checks passed
github-actions bot pushed a commit that referenced this pull request Feb 5, 2026
github-actions bot pushed a commit that referenced this pull request Feb 5, 2026
pquentin added a commit that referenced this pull request Feb 5, 2026
(cherry picked from commit 96706df)

Co-authored-by: Quentin Pradet <quentin.pradet@elastic.co>
pquentin added a commit that referenced this pull request Feb 6, 2026
(cherry picked from commit 96706df)

Co-authored-by: Quentin Pradet <quentin.pradet@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants