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

Version compatibility bits (remove old compat code, bump requirement to 3.7+) #767

Merged
merged 4 commits into from
Dec 20, 2023

Conversation

akx
Copy link
Contributor

@akx akx commented Dec 20, 2023

With #757 and 0bc4d41, it's clear that this library is not meant to support Pythons older than 3.7, so:

  • remove unnecessary Python version check bits
  • add the requires_python stanza to setup.py to mark this as compatible with Python 3.7+.

@akx
Copy link
Contributor Author

akx commented Dec 20, 2023

@PierreF I see you pushed 0bc4d41 to master – does this mean we should target Python 3.7+?

@PierreF
Copy link
Contributor

PierreF commented Dec 20, 2023

Yes, I'm dropping support for Python 2 which cause trouble and Python 2 is EOL for long time. For that reason the next release I'll do is a 2.0 so we could do some breaking changes. Dropping support for Python 3.5 and 3.6 seems a good idea.

I've kept 3.7 because is was EOL only few months ago.

@PierreF
Copy link
Contributor

PierreF commented Dec 20, 2023

BTW: could you add a Signed-Off to your commit ? This is a requirement from Eclipse to acknowledge you agree with the ECA.

@akx
Copy link
Contributor Author

akx commented Dec 20, 2023

@PierreF Added signoffs and bumped the references to 3.6 to 3.7.

@akx akx changed the title Version compatibility bits Version compatibility bits (remove old compat code, bump requirement to 3.7+) Dec 20, 2023
@PierreF PierreF merged commit a2d01de into eclipse-paho:master Dec 20, 2023
5 checks passed
@akx akx deleted the version-compat-bits branch December 20, 2023 19:18
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