Fix to remove python 2 & python 3 switches using if statements by vgali7 · Pull Request #1168 · apache/cassandra-python-driver · GitHub
Skip to content

Fix to remove python 2 & python 3 switches using if statements#1168

Closed
vgali7 wants to merge 5 commits into
apache:masterfrom
vgali7:master
Closed

Fix to remove python 2 & python 3 switches using if statements#1168
vgali7 wants to merge 5 commits into
apache:masterfrom
vgali7:master

Conversation

@vgali7

@vgali7 vgali7 commented Jun 26, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@bschoening

Copy link
Copy Markdown
Contributor

@absurdfarce

Copy link
Copy Markdown
Contributor

Greetings @vgali7 and thanks for the contribution! I restarted the test run as requested by @bschoening; we'll see if we can reliably reproduce the test failures we saw before.

Have you signed the Contributor License Agreement for contributions to DataStax open source projects? If not you can find it at https://cla.datastax.com/. Thanks!

@bschoening

bschoening commented Jul 6, 2023

Copy link
Copy Markdown
Contributor

Updated appveyor.yml to Python 3.7

@vgali7

vgali7 commented Jul 10, 2023

Copy link
Copy Markdown
Contributor Author

@absurdfarce Thanks for the tip, I've signed the Contributor License Agreement!

@vgali7

vgali7 commented Jul 19, 2023

Copy link
Copy Markdown
Contributor Author

@absurdfarce Hey I just wanted to check that this looks good and is ready to commit, thanks! I have some further updates to remove obsolete 'six' compatibility but wanted to get this committed first.

@absurdfarce

Copy link
Copy Markdown
Contributor

Hey @vgali7, apologies for not getting back to you sooner. We're in a rush to address some end-of-quarter updates so I haven't taken a look at this in detail yet. I should have more time to look it over in the early part of August.

Thanks for your patience!

@absurdfarce

absurdfarce commented Oct 9, 2023

Copy link
Copy Markdown
Contributor

@absurdfarce absurdfarce closed this Oct 9, 2023
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