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

Remove duplicate license parameter #180

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Remove duplicate license parameter #180

wants to merge 1 commit into from

Conversation

blag
Copy link

@blag blag commented Nov 7, 2019

The Python packaging specifications indicate that the license parameter to setup() should be used in one of the following cases:

  • the license is not a selection from the “License” Trove classifiers
  • to specify a particular version of a license which is named via the Classifier field
  • to indicate a variation or exception to such a license

This PR removes the explicit license parameter since the usage for this project does not fit any of those cases and it makes it more difficult to use tools like pip-licenses since it is not consistent with other projects that just have License :: OSI Approved :: MIT License as one of their trove classifiers.

Please note that this does not change the license of the project in any way, and since this project already has a License :: OSI Approved :: MIT License trove classifier the license parameter is redundant.

@blag
Copy link
Author

blag commented Nov 7, 2019

The Travis CI failures are due to old versions of Python, not due to this change.

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.

1 participant