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

Add python 2/3 trove classifiers #13

Merged
merged 1 commit into from
Nov 23, 2019
Merged

Add python 2/3 trove classifiers #13

merged 1 commit into from
Nov 23, 2019

Conversation

proinsias
Copy link
Contributor

Clearly identify this project as supporting python 2 and 3.
This is useful for utility programs like caniusepython3.

@CAM-Gerlach
Copy link

@lidaobing Any luck reviewing this? The well-known Py3Readiness site is considering an increase in the number of packages displayed in chhantyal/py3readiness#40 (see preview) which would result in this package, one of the 1000 most downloaded on PyPI, to be included within the few remaining packages that do not support Python 3 when it in fact does just fine. Thanks!

Copy link

@hugovk hugovk left a comment

Choose a reason for hiding this comment

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

+1, this makes it clear on PyPI what this project supports.

Couple of little suggestions.

setup.py Outdated Show resolved Hide resolved
setup.py Show resolved Hide resolved
@CAM-Gerlach
Copy link

@proinsias To maximize the changes of this getting merged as is, I'd recommend just squashing (fixup-ing) the two tiny fixes into your first commit, since they are so trivial, have rather unclear commit messages and (at least for the whitespace) fix a minor style issue induced by the original commit).

Clearly identify this project as supporting python 2 and 3.
This is useful for utility programs like
[caniusepython3](https://github.com/brettcannon/caniusepython3#how-do-you-tell-if-a-project-has-been-ported-to-python-3).

Includes changes suggested by Hugo van Kemenade <hugovk@users.noreply.github.com>.
@lidaobing lidaobing merged commit 397b4a0 into lidaobing:master Nov 23, 2019
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.

4 participants