Skip to content

chode(multiple samples) Updating django instances and some of it dependencies.#14201

Draft
XrossFox wants to merge 1 commit into
GoogleCloudPlatform:mainfrom
XrossFox:python-django-dependency-update-1
Draft

chode(multiple samples) Updating django instances and some of it dependencies.#14201
XrossFox wants to merge 1 commit into
GoogleCloudPlatform:mainfrom
XrossFox:python-django-dependency-update-1

Conversation

@XrossFox
Copy link
Copy Markdown
Contributor

@XrossFox XrossFox commented May 20, 2026

WIP

Note: run/django is not working properly, this is a known bug documented in b/479621565

Description

Fixes #

Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.

Checklist

@product-auto-label product-auto-label Bot added samples Issues that are directly related to samples. api: appengine Issues related to the App Engine Admin API API. api: container Issues related to the Kubernetes Engine API API. api: run Issues related to the Cloud Run API. labels May 20, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request attempts to update dependencies across several projects, including Django, django-environ, and pytest-django. The reviewer identified that the requested versions (Django 6.0.5, django-environ 0.13.0, and pytest-django 4.12.0) are not valid releases on PyPI and suggests reverting to the latest stable versions (Django 5.1.6, django-environ 0.12.0, and pytest-django 4.10.0) while maintaining compatibility with supported Python runtimes.

Comment thread appengine/flexible/django_cloudsql/requirements.txt
Comment thread appengine/flexible/django_cloudsql/requirements.txt
Comment thread appengine/flexible/django_cloudsql/requirements-test.txt
Comment thread run/django/requirements-test.txt
@XrossFox
Copy link
Copy Markdown
Contributor Author

Code Review

This pull request attempts to update dependencies across several projects, including Django, django-environ, and pytest-django. The reviewer identified that the requested versions (Django 6.0.5, django-environ 0.13.0, and pytest-django 4.12.0) are not valid releases on PyPI and suggests reverting to the latest stable versions (Django 5.1.6, django-environ 0.12.0, and pytest-django 4.10.0) while maintaining compatibility with supported Python runtimes.

This is not true at all, i've checked existence directly in Pypi. The point of this PR is to update to secure versions of Django (6.0.5), which no longer support any python release under 3.12.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: appengine Issues related to the App Engine Admin API API. api: container Issues related to the Kubernetes Engine API API. api: run Issues related to the Cloud Run API. samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant