Skip to content

chore(django-google-spanner): update Python version to 3.10 - #16914

Merged
chalmerlowe merged 4 commits into
mainfrom
chore/update-python-version-django-google-spanner
May 5, 2026
Merged

chore(django-google-spanner): update Python version to 3.10#16914
chalmerlowe merged 4 commits into
mainfrom
chore/update-python-version-django-google-spanner

Conversation

@chalmerlowe

Copy link
Copy Markdown
Contributor

This PR updates django-google-spanner to establish Python 3.10 as the minimum supported version, dropping support for Python 3.9 and below.

Changes

  • Configuration: Updated setup.py and noxfile.py to require Python 3.10+ and remove references to Python 3.7, 3.8, and 3.9.
  • Cleanup: Removed dead code in noxfile.py that was skipping Python 3.7.

Blocking

This PR is blocked by PR #16910

@chalmerlowe
chalmerlowe requested a review from a team as a code ownerMay 1, 2026 13:28

@gemini-code-assistgemini-code-assistBot left a comment

Copy link
Copy Markdown
Contributor

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 removes support for Python 3.7, 3.8, and 3.9 across the project's configuration files, updating the minimum required version to 3.10 in both noxfile.py and setup.py. Feedback suggests adding PyPI classifiers for Python 3.13 and 3.14 to setup.py to ensure the metadata is consistent with the versions currently being tested in the Nox configuration.

I am having trouble creating individual review comments. Click here to see my feedback.

packages/django-google-spanner/setup.py (76)

medium

The noxfile.py configuration indicates that Python 3.13 and 3.14 are supported (though 3.14 is currently skipped due to Protobuf). To ensure consistency and proper indexing on PyPI, please add the corresponding classifiers for these versions in the classifiers list above.

Comment on lines -126 to -127
if session.python == "3.9":
session.skip("Python 3.9 is not supported for Django 5.2 tests")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

nit: we should keep this until 3.9 is removed from gapic-generator and autogenerated packages which account for 90% of the packages in this repo.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

This PR and one's similar to it are blocked and are only presented for discussion.
Once the gapic-generator is updated fully to 3.10, these will be unblocked.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

PR 16910 has been merged. 3.9 is no longer supported in the gapic generator AND we have instituted logic in the run_single_test.sh script to only test unit-3.9 nox sessions in packages that still support it.

@parthea

Copy link
Copy Markdown
Contributor

Switching to draft until presubmits are green

@parthea
parthea marked this pull request as draft May 1, 2026 15:45
@chalmerlowechalmerlowe self-assigned this May 4, 2026
@chalmerlowechalmerlowe added kokoro:force-run Add this label to force Kokoro to re-run the tests. and removed kokoro:force-run Add this label to force Kokoro to re-run the tests. labels May 4, 2026
Comment threadpackages/django-google-spanner/noxfile.py Outdated
Kokoro is being wonky. Using this to manually trigger CI/CD since kokoro force-run label is not working right now.
@chalmerlowe
chalmerlowe marked this pull request as ready for review May 4, 2026 22:29
@chalmerlowe
chalmerlowe merged commit 1c4a3d9 into mainMay 5, 2026
47 checks passed
@chalmerlowe
chalmerlowe deleted the chore/update-python-version-django-google-spanner branch May 5, 2026 08:09
zhixiangli pushed a commit to zhixiangli/google-cloud-python that referenced this pull request May 6, 2026
…is#16914)
This PR updates `django-google-spanner` to establish Python 3.10 as the
minimum supported version, dropping support for Python 3.9 and below.
### Changes
* Configuration: Updated `setup.py` and `noxfile.py` to require Python
3.10+ and remove references to Python 3.7, 3.8, and 3.9.
* Cleanup: Removed dead code in `noxfile.py` that was skipping Python
3.7.
### Blocking
This PR is blocked by PR googleapis#16910
Sign up for freeto 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.

3 participants

@chalmerlowe@parthea@ohmayr