Skip to content

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.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 76.593% when pulling c17ec86 on blag:patch-1 into 667703f on diyan:master.

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