-
Notifications
You must be signed in to change notification settings - Fork 373
Add support for Python 3.13 in templates, CI, and codebase #1364
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
Conversation
You need to install
And then re-commit. It's failing on the linter. |
Linting fixed here @RemyTinco |
Co-authored-by: Graham Lyon <[email protected]>
Fix lint error in test
I successfully deployed a Django REST Framework (DRF) project on Python 3.13 using this branch of Zappa. Installed the branch locally with pip:
Here’s the project repo I used for testing: https://github.com/RemyTinco/drf-zappa-poc-313 The app is live here: https://6fmlmc1r2g.execute-api.eu-west-2.amazonaws.com/dev/api/version/ Here’s the JSON response from the
|
Hey @monkut, all the code changes are in and the test deployment to AWS with Python 3.13 looks good (shared above). I'm just waiting on the CI checks to start/finish. In the meantime, happy to receive any review or thoughts on next steps whenever you have a moment! |
I Can attest to Remy. I have deployed my app TallyOnWheels with Python 3.13 without any hiccups. All lamda invocation seems to be happening without any issue. |
Looks like the version of django used in the testcase requires |
Sorry about the delays, we've had some org/project issues that recently got sorted out. |
🔨 add legacy-cgi to dev-packages to resolve testcase FAILURE: FAILED tests/tests_utilities.py::GeneralUtilitiesTestCase::test_dj_wsgi - ModuleNotFoundError: No module named 'cgi' :arrow_up: update pip min version dependency :arrow_up: update requests min version dependency :arrow_up: update tqdm min version dependency :arrow_up: update django min version dev dependency :arrow_up: update black min version dev dependency
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
This PR adds support for Python 3.13 in Zappa, enabling deployments on AWS Lambda with the latest Python runtime.
The following updates have been made:
SUPPORTED_VERSIONS
inzappa/__init__.py
to include Python 3.13.get_runtime_from_python_version()
inzappa/utilities.py
..github/workflows/ci.yml
) to test against Python 3.13.setup.py
classifiers to reflect Python 3.13 support.README.md
and issue templates to mention Python 3.13 compatibility.tests/tests.py
for verifying Python 3.13 runtime compatibility.Testing
This PR was tested by:
🟧 Running CI tests across supported Python versions (3.8 – 3.13).
🟩 Deploying a test Zappa application with Python 3.13 to AWS Lambda.
🟩 : Running unit tests to verify manylinux wheel compatibility with Python 3.13.
GitHub Issue Reference
Closes #1363