Skip to content
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

deps: Remove Python 3.7 and 3.8 as supported runtimes #2133

Merged
merged 67 commits into from
Feb 28, 2025

Conversation

chalmerlowe
Copy link
Collaborator

@chalmerlowe chalmerlowe commented Feb 14, 2025

Fix: Update Python versions by dropping support for versions 3.7, 3.8.

Also includes minor fixes and tweaks needed to support the primary changes related to dropping runtimes.

This is a work in progress. Do not merge.

@chalmerlowe chalmerlowe requested review from a team as code owners February 14, 2025 15:29
@product-auto-label product-auto-label bot added size: l Pull request size is large. api: bigquery Issues related to the googleapis/python-bigquery API. labels Feb 14, 2025
@chalmerlowe chalmerlowe removed the request for review from Neenu1995 February 14, 2025 15:29
@chalmerlowe chalmerlowe added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Feb 14, 2025
@chalmerlowe chalmerlowe requested a review from a team as a code owner February 14, 2025 16:29
@chalmerlowe chalmerlowe requested a review from nicain February 14, 2025 16:29
gcf-owl-bot bot and others added 17 commits February 14, 2025 16:31
@@ -33,7 +33,6 @@ export PYTHONUNBUFFERED=1
env | grep KOKORO

# Install nox
# `virtualenv==20.26.6` is added for Python 3.7 compatibility
Copy link
Contributor

Choose a reason for hiding this comment

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

should we remove virtualenv==20.26.6 because 3.7 is removed too?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We want virtualenv.
I am testing locally to see if we can simply install virtualenv without specifying a version number.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Somewhere Owlbot.py did some post processing and reverted that bit of code to add that change back in because it pulled in the template for certain files and overwrote my changes.
I edited owlbot.py so that it will not touch certain templates AND so that for certain templates that it will protect the edits that we need to have go through.

@chalmerlowe chalmerlowe added owlbot:run Add this label to trigger the Owlbot post processor. and removed owlbot:ignore instruct owl-bot to ignore a PR labels Feb 28, 2025
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Feb 28, 2025
Copy link
Contributor

@Linchin Linchin left a comment

Choose a reason for hiding this comment

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

Thank you so much for working on deprecating 3.7 and 3.8, it's been great work! LGTM. 🎉

@chalmerlowe chalmerlowe merged commit fb7de39 into main Feb 28, 2025
17 of 23 checks passed
@chalmerlowe chalmerlowe deleted the fix-b387301099-remove-python-37 branch February 28, 2025 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigquery Issues related to the googleapis/python-bigquery API. size: l Pull request size is large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants