upgrade python and fastapi#60
Conversation
There was a problem hiding this comment.
Pull request overview
Upgrades the project’s Python runtime baseline and core web stack dependencies to address upcoming Python 3.10 EOL and unblock newer transitive dependency versions (e.g., Starlette/idna) for security alert remediation.
Changes:
- Bump supported Python version to 3.11 (project metadata + CI workflow).
- Upgrade FastAPI and related pinned dependencies in exported
requirements.txt. - Increment project version to
0.11.4.
Reviewed changes
Copilot reviewed 2 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| requirements.txt | Updates pinned runtime dependency set produced by uv export, including FastAPI and transitive packages. |
| pyproject.toml | Raises requires-python to 3.11+, bumps project version, and updates the FastAPI dependency constraint. |
| .github/workflows/python-package.yml | Updates CI to run using Python 3.11. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| dependencies = [ | ||
| "fastapi (>=0.116.0)", | ||
| "fastapi>=0.137.2", | ||
| "pydantic-settings (>=2.7.1,<3.0.0)", | ||
| "sqlalchemy (>=2.0.36,<3.0.0)", | ||
| "jinja2 (>=3.1.5,<4.0.0)", |
datadavev
left a comment
There was a problem hiding this comment.
Looks OK - the version changes don't appear to create any conflict or other issues.
sfisher
left a comment
There was a problem hiding this comment.
This looks ok to me though I don't know if I tested thoroughly.
I was able to update, (uv sync I think?) and to start the application (though I think maybe I'm missing something or it doesn't like http?) and I was able to run tests successfully (uv run pytest).
I'm sure Dave knows better than me, but it seems to be probably working fine.
@datadavev @sfisher Hi Dave and Scott,
Here are the major changes:
Please review and let me know if have questions.
Thank you
Jing