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

Remove Python 2 support, six dependency #619

Merged
merged 1 commit into from
Oct 16, 2022

Conversation

rdb
Copy link
Contributor

@rdb rdb commented Mar 12, 2022

Thanks for this great project!

Python 2 is ancient history by now, and as pointed out in #618, pyjnius does not even work with Python versions under 3.6 at this point.

This PR removes the compatibility hacks for Python 2 and drops the now unnecessary dependency on the six package.

@misl6 misl6 added this to the 1.5.0 milestone Jul 3, 2022
@misl6
Copy link
Member

misl6 commented Jul 3, 2022

We're in the process of releasing version 1.4.2, which will be a bugfix one.
As this one will break Python 2 compatibility, I've added it to milestone 1.5.0.

@rdb We now have some conflicts, can you take care of it?

@rdb rdb force-pushed the remove-py2-support branch from f3b7899 to 95e3540 Compare September 6, 2022 10:02
@rdb
Copy link
Contributor Author

rdb commented Sep 6, 2022

Sorry for the delay, I've addressed the merge conflicts and rebased. Tests pass locally on Linux.

As this one will break Python 2 compatibility, I've added it to milestone 1.5.0.

This totally makes sense and is no problem for me, just wanted to point out that no Python 2 compatibility is broken by this PR, since pyjnius is already incompatible with Python 2 due to the use of f-strings :)

Copy link
Member

@misl6 misl6 left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

@misl6 misl6 merged commit ee4e9c2 into kivy:master Oct 16, 2022
@rdb rdb deleted the remove-py2-support branch October 16, 2022 13:17
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.

2 participants