Skip to content

⬆️ Update anyio version in test suite#13128

Closed
graingert wants to merge 6 commits into
fastapi:masterfrom
graingert:patch-3
Closed

⬆️ Update anyio version in test suite#13128
graingert wants to merge 6 commits into
fastapi:masterfrom
graingert:patch-3

Conversation

@graingert

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread requirements-tests.txt Outdated
graingert and others added 2 commits January 1, 2025 12:00
Co-authored-by: Marcelo Trylesinski <marcelotryle@gmail.com>
Comment thread requirements-tests.txt Outdated
@graingert graingert marked this pull request as ready for review January 22, 2025 18:50
@graingert

Copy link
Copy Markdown
Contributor Author

@tiangolo hmm python 3.8 is no longer supported, and no longer supported by starlette. What would you like me to do:

  • Want me to only bump the anyio dep for 3.9+
  • Ignore the warning in 3.8
  • Drop support for EOL python 3.8 in FastAPI?

@svlandeg

Copy link
Copy Markdown
Member

Hi @graingert! In general, we do try to keep support for older Python versions, even when they're EOL, as long as it doesn't become too burdensome on our end (e.g. issues with CI, etc). In this case, I do think we'll want to continue support for Python 3.8 at least for a bit longer. As such, only bumping for 3.9+ would make the most sense to me.

@svlandeg

Copy link
Copy Markdown
Member

Just as a maintenance note: I'll put this PR in draft as long as the test suite is failing. Feel free to mark it as "Ready for review" once you're done and everything is green! 🙏

@svlandeg svlandeg marked this pull request as draft February 18, 2025 15:18
@svlandeg svlandeg changed the title Update anyio in requirements-tests.txt ⬆️ Update anyio version in test suite Feb 18, 2025
@github-actions github-actions Bot added the conflicts Automatically generated when a PR has a merge conflict label Sep 5, 2025
@github-actions

github-actions Bot commented Sep 5, 2025

Copy link
Copy Markdown
Contributor

This pull request has a merge conflict that needs to be resolved.

@svlandeg svlandeg self-assigned this Sep 16, 2025

@svlandeg svlandeg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This PR has become obsolete after #13273 has bumped the range to >=3.2.1,<5.0.0. Thanks again for working on this! 🙏

@svlandeg svlandeg closed this Sep 16, 2025
@svlandeg svlandeg removed their assignment Sep 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conflicts Automatically generated when a PR has a merge conflict internal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants