Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat!: Remove Python 2 support #657

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

Merged
merged 13 commits into from
Jan 12, 2022
Merged

feat!: Remove Python 2 support #657

merged 13 commits into from
Jan 12, 2022

Conversation

andrewsg
Copy link
Contributor

BREAKING CHANGE: This PR removes Python 2 support in the setup.py metadata, test runner and code itself. This is intended to be released as a major version update. Python 2 users can continue to use the 1.x version of the client library.

@andrewsg andrewsg added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 22, 2021
@andrewsg andrewsg requested review from a team as code owners November 22, 2021 23:14
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Nov 22, 2021
@product-auto-label product-auto-label bot added the api: storage Issues related to the googleapis/python-storage API. label Nov 22, 2021
@tseaver
Copy link
Contributor

tseaver commented Nov 26, 2021

@andrewsg The Kokoro system-2.7 job is configured both in the .kokoro/presubmit/system-2.7.cfg file and in google3.

@andrewsg
Copy link
Contributor Author

andrewsg commented Jan 7, 2022

@tseaver This was on hold due to the release freeze last month. Thank you as always for your thorough review. PTAL. Will address google3 issue shortly.

@parthea parthea requested a review from tseaver January 8, 2022 11:18
@unforced
Copy link
Contributor

unforced commented Jan 11, 2022

So happy to see all the clean up. Nothing else to add beyond the requests that tres and cathy made; lgtm. Is there a change that should be made in here for kokoro to not try and still run the system-2.7 tests?
EDIT: Oh seems you already did.

Co-authored-by: cojenco <[email protected]>
Copy link
Contributor

@cojenco cojenco left a comment

Choose a reason for hiding this comment

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

LGTM🎉🎉 Thanks Andrew for getting us across the finish line!

@andrewsg andrewsg removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jan 12, 2022
@andrewsg andrewsg merged commit b611670 into main Jan 12, 2022
@andrewsg andrewsg deleted the python-2-turndown branch January 12, 2022 18:12
This was referenced Jan 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: storage Issues related to the googleapis/python-storage API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants