Skip to content

Conversation

@fladi
Copy link
Contributor

@fladi fladi commented Feb 18, 2016

There is no need to install the LICENSE file. It would end up in a global
namespace, possibly overwriting an identically named file there.

@owais
Copy link
Collaborator

owais commented Feb 21, 2016

Let's update to latest master or include in your MANIFEST.in PR

@fladi fladi force-pushed the no_license_as_datafile branch from cdb3a52 to 65d9cdd Compare February 25, 2016 07:44
@fladi
Copy link
Contributor Author

fladi commented Feb 25, 2016

@owais Rebased.

@owais
Copy link
Collaborator

owais commented Feb 25, 2016

Looks like there are still conflicts

There is no need to install the LICENSE file. It would end up in a global
namespace, possibly overwriting an identically named file there.
@fladi fladi force-pushed the no_license_as_datafile branch from 65d9cdd to 45793c4 Compare February 25, 2016 08:54
@fladi
Copy link
Contributor Author

fladi commented Feb 25, 2016

@owais Merging PR #47 cuased this. I rebased again.

owais added a commit that referenced this pull request Feb 25, 2016
Do not install LICENsE as a datafile.
@owais owais merged commit 8e9192d into django-webpack:master Feb 25, 2016
@fladi fladi deleted the no_license_as_datafile branch February 25, 2016 12:45
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