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

setup.py: python_requires #179

Merged
merged 1 commit into from
Jun 22, 2020
Merged

Conversation

singingwolfboy
Copy link
Contributor

Since this package is only compatible with Python 3, we should use the python_requires argument in setup.py so that tools like pip don't try to install it on unsupported versions of Python.

https://packaging.python.org/guides/distributing-packages-using-setuptools/#python-requires

All other changes were automatically made by black in my editor. I can revert them if you prefer.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 79.644% when pulling 2e565cc on singingwolfboy:python-requires into 671ad91 on sh4nks:master.

@sh4nks sh4nks merged commit 062401f into pallets-eco:master Jun 22, 2020
@sh4nks
Copy link
Collaborator

sh4nks commented Jun 22, 2020

Thanks for your PR!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants