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

docs(LICENSE): relicense to JSF #66

Merged
merged 2 commits into from
Feb 22, 2017
Merged

docs(LICENSE): relicense to JSF #66

merged 2 commits into from
Feb 22, 2017

Conversation

michael-ciniawsky
Copy link
Member

@webpack/core-team Docs incoming 😛

@jhnns
Copy link
Member

jhnns commented Feb 21, 2017

Thx 👍

Can you also adjust the license field in the package.json? 😁

@michael-ciniawsky
Copy link
Member Author

@jhnns Yep sure 😛

@michael-ciniawsky
Copy link
Member Author

@jhnns done, I also removed LICENSE && README from pkg.files since they get published anyways by npm defaults

@jhnns jhnns merged commit 19bd9ee into webpack:master Feb 22, 2017
@jhnns
Copy link
Member

jhnns commented Feb 22, 2017

I also removed LICENSE && README from pkg.files since they get published anyways by npm defaults

I thought that too, but I've added them because with loader-utils 0.2.17, npmjs.org was saying that there is no README. So I'll re-add it.

@michael-ciniawsky
Copy link
Member Author

@jhnns Uhh 😧, that's weird normally it should work, sry fumbling around 🤗

@michael-ciniawsky michael-ciniawsky deleted the license branch February 22, 2017 21:43
@whitelynx
Copy link

Why was the license field in package.json changed? According to the NPM docs the license field should be set to something from the SPDX license list, but JSF does not appear there. Since this was only a reassignment of copyright, the license field should still stay MIT. (it is still the MIT license, just with a different copyright assignee now)

@jhnns
Copy link
Member

jhnns commented Mar 10, 2017

Yes, you're right. We need to change that back 😁

@mlucool
Copy link

mlucool commented Mar 13, 2017

@jhnns Can this be republished?

@jhnns
Copy link
Member

jhnns commented Mar 14, 2017

Published as 1.0.4

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.

5 participants