Skip to content

Add minimum python version in setup.py #93

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

Merged
merged 1 commit into from
Oct 9, 2021

Conversation

Lothiraldan
Copy link
Contributor

It looks like latest released version don't support Python 3.5 anymore. It is
failing on line
https://github.com/Pithikos/python-websocket-server/blob/master/websocket_server/websocket_server.py#L296
because it is using a f-string which is only available in Python 3.6+.

If it was intentional, this PR is adding the correct package metadata so pip
can choose the right version depending on the user Python Version.

It looks like latest released version don't support Python 3.5 anymore. It is
failing on line
https://github.com/Pithikos/python-websocket-server/blob/master/websocket_server/websocket_server.py#L296
because it is using a f-string which is only available in Python 3.6+.

If it was intentional, this PR is adding the correct package metadata so pip
can choose the right version depending on the user Python Version.
@Pithikos
Copy link
Owner

Pithikos commented Oct 9, 2021

Good catch. Thank you!

@Pithikos Pithikos merged commit fe15768 into Pithikos:master Oct 9, 2021
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