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

Update PythonTest.sh #8393

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update PythonTest.sh #8393

wants to merge 1 commit into from

Conversation

TJKoury
Copy link
Contributor

@TJKoury TJKoury commented Sep 6, 2024

Allow tests to complete as Python 2 does not support type-hinting, drop support for Python 2 per #6329.

Allow tests to complete as Python 2 does not support type-hinting, drop support for Python 2 per google#6329.
@TJKoury
Copy link
Contributor Author

TJKoury commented Dec 23, 2024

@aardappel

@aardappel
Copy link
Collaborator

Unlike 4 years ago, I think we'd be fine dropping Python2 support at this point, wdyt @dbaileychess

run_tests python2.6 100 100 100 100 false
run_tests python2.7 100 100 100 100 false
run_tests python2.7 100 100 100 100 true
# run_tests python2.6 100 100 100 100 false
Copy link
Contributor

Choose a reason for hiding this comment

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

i guess if it is decided to remove Python2 support this can be deleted and not just commented out

Sign up for free to 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