Skip to content

upgrade python and fastapi#60

Merged
jsjiang merged 1 commit into
mainfrom
57_upgrade_fastapi
Jun 23, 2026
Merged

upgrade python and fastapi#60
jsjiang merged 1 commit into
mainfrom
57_upgrade_fastapi

Conversation

@jsjiang

@jsjiang jsjiang commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

@datadavev @sfisher Hi Dave and Scott,
Here are the major changes:

  • Upgrade Python to 3.11 as 3.10 will be end of life 2026-10
  • Upgrade fastapi to >=0.116.0 so packages such as starlette and idna can be updated to newer versions to fix Dependabot security alerts.

Please review and let me know if have questions.

Thank you

Jing

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

Comment thread pyproject.toml
Comment on lines 9 to 13
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 datadavev left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks OK - the version changes don't appear to create any conflict or other issues.

@sfisher sfisher left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

@jsjiang jsjiang merged commit 9eceb08 into main Jun 23, 2026
2 checks passed
This was referenced Jun 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants