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 LICENSE.md to the built wheel #4270

Merged
merged 1 commit into from Jul 18, 2016
Merged

Add LICENSE.md to the built wheel #4270

merged 1 commit into from Jul 18, 2016

Conversation

pombredanne
Copy link
Contributor

@pombredanne pombredanne commented Jul 15, 2016

The wheel released on Pypi does not include the license file . The way to do this is by updating the setup.cfg accordingly. This fix will ensure that the LICENSE.md file is available in the dist-info dir of the built wheels as LICENSE.txt.
This way folks redistribution an official DRF wheel are complying with the BSD license... otherwise they would not by virtue of not including the license text!

The wheel released on Pypi does not include the license file .
The way to do this is by updating the setup.cfg accordingly
@codecov-io
Copy link

codecov-io commented Jul 15, 2016

Current coverage is 91.18%

Merging #4270 into master will not change coverage

@@             master      #4270   diff @@
==========================================
  Files            52         52          
  Lines          5776       5776          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           5267       5267          
  Misses          509        509          
  Partials          0          0          

Powered by Codecov. Last updated by 91aad92...e1eb420

@tomchristie
Copy link
Member

tomchristie commented Jul 18, 2016

Great, thanks for this!

@tomchristie tomchristie merged commit e476c22 into encode:master Jul 18, 2016
2 checks passed
@tomchristie tomchristie added this to the 3.4.1 Release milestone Jul 18, 2016
@pombredanne
Copy link
Contributor Author

pombredanne commented Aug 1, 2016

@tomchristie Thannks!

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

Successfully merging this pull request may close these issues.

None yet

4 participants