Skip to content
This repository has been archived by the owner on Jun 12, 2022. It is now read-only.

fix(license): switch to ISC #49

Merged
merged 1 commit into from
Mar 12, 2018
Merged

fix(license): switch to ISC #49

merged 1 commit into from
Mar 12, 2018

Conversation

zkat
Copy link
Owner

@zkat zkat commented Nov 15, 2017

As part of the ongoing relicensing process, this changes
the license over to one more palatable to open source
consumers.

BREAKING CHANGE: license changed from CC0 to ISC.

NOTE

This requires consent from all contributors. If you're on the list below, please reply with "I agree" or a 👍 or something similar to let me know you're ok with your changes being included in this relicensing.

The reason this is happening is because it's pretty hard for some folks to use CC0 stuff, since it's not an OSI-approved license. ISC is the license used by the npm CLI, so I'm switching to it for convenience. We ran all this through our company lawyer before going about it, too, and it's part of a number of relicensing PRs that are happening to related projects I've worked on! Let me know if you have any questions!

As part of the ongoing relicensing process, this changes
the license over to one more palatable to open source
consumers.

BREAKING CHANGE: license changed from CC0 to ISC.
@yyjdelete
Copy link
Contributor

👍

1 similar comment
@nlkluth
Copy link
Contributor

nlkluth commented Nov 15, 2017

👍

@bitinn
Copy link
Contributor

bitinn commented Nov 15, 2017 via email

@zkat
Copy link
Owner Author

zkat commented Nov 15, 2017

@bitinn I can't change the license on node-fetch-npm, so that stays MIT. This applies solely to the m-f-h codebase, which doesn't share code with node-fetch. In order to change the license for node-fetch-npm, I would require permission from all contributors of that project. MIT is perfectly fine, though, and has basically the same benefits as ISC in this context, so I wouldn't even want to do that.

Since node-fetch is MIT, that also means it's not required for dependents to adopt that license.

@colinrotherham
Copy link
Contributor

👍

@bitinn
Copy link
Contributor

bitinn commented Nov 15, 2017 via email

@SimenB
Copy link
Contributor

SimenB commented Nov 15, 2017

👍

@zkat
Copy link
Owner Author

zkat commented Nov 15, 2017

@bitinn ahh, that was me referencing similar relicensing efforts for cacache, pacote, and the other npm5-related libraries, which I was defaulting to CC0 before realizing the issues that would cause in practice (I'd been using that license for years).

@miiihi
Copy link
Contributor

miiihi commented Nov 15, 2017 via email

@cilice
Copy link
Contributor

cilice commented Mar 8, 2018

👍

1 similar comment
@tmyt
Copy link
Contributor

tmyt commented Mar 9, 2018

👍

@zkat zkat merged commit bf90c6d into latest Mar 12, 2018
@zkat zkat deleted the zkat/relicense branch March 12, 2018 21:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants