-
Notifications
You must be signed in to change notification settings - Fork 313
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
Conversation
…googleapis/python-bigquery into fix-b387301099-remove-python-37
…googleapis/python-bigquery into fix-b387301099-remove-python-37
@@ -33,7 +33,6 @@ export PYTHONUNBUFFERED=1 | |||
env | grep KOKORO | |||
|
|||
# Install nox | |||
# `virtualenv==20.26.6` is added for Python 3.7 compatibility |
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.
should we remove virtualenv==20.26.6
because 3.7 is removed too?
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.
We want virtualenv.
I am testing locally to see if we can simply install virtualenv without specifying a version number.
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.
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.
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.
Thank you so much for working on deprecating 3.7 and 3.8, it's been great work! LGTM. 🎉
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.