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 supported Python versions and add a badge in the README #530

Merged
merged 2 commits into from
Sep 21, 2021

Conversation

maximlt
Copy link
Member

@maximlt maximlt commented Sep 20, 2021

The Python versions supported by param are reflected by those currently tested in the CI, i.e. 2.7, 3.6, 3.7, 3.8 and 3.9. This PR updates setup.py to list these versions. Note that Python 3.10 will be out soon and it'd be nice to add it to the CI and see how that goes.

This PR also adds a badge to the README that shows the Python versions supported by param. Since param is meant to be a foundational library of its kind, it's nice to show that it is compatible with many versions of Python. This badge could also be added to the landing page of the site if required.

@jbednar jbednar merged commit 9384828 into master Sep 21, 2021
@jbednar jbednar deleted the update_setuppy_and_badge branch September 21, 2021 15:01
@MarcSkovMadsen
Copy link
Collaborator

I would get 2.7 out. It could signal that Param is legacy, out of date or no longer actively supported. I.e. give a wrong signal.

@jbednar
Copy link
Member

jbednar commented Sep 21, 2021

Param is a foundational library and thus should be more open in version support than just about any other library. But like Python 2.7 itself, I think that concern only applies to Param 1.x, which should continue to be usable with Python 2.7 indefinitely. Once we switch to Param 2.x I don't see any reason to provide Python 2.7 support any longer.

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