This repository was archived by the owner on Jul 28, 2021. It is now read-only.
meta: relicense to Artistic-2.0 #22
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Primarily for compatibility with the main npm codebase, to simplify things in the future
Contributor
|
Reviewed the |
Contributor
|
+1 |
Contributor
|
I'm cool with the Artistic License. |
Contributor
|
Ship it! 🚢 |
Contributor
|
I agree with the relicensing to Artistic-2.0 License |
Contributor
|
I dislike that the License text is a lot longer, but I should not be stopping this. |
Contributor
|
Sure |
Contributor
|
@zkat, ship it! 👍 |
Contributor
|
Sure
Alejandro López <notifications@github.com> 于 2018年11月10日周六 18:53写道：
… @zkat <https://github.com/zkat>, ship it! 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#22 (comment)>, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/AB49L6ClmK2cpY4fx6c3zsFaJQyMFXy2ks5utrAvgaJpZM4YXPS0>
.
|
Contributor
|
For sure. |
Contributor
|
I agree. |
|
I agree! |
Contributor
Author
|
Thanks, everyone! I'll be merging this now. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
13 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Primarily for compatibility with the main npm codebase, to simplify things in the future. The Artistic License is a permissive license and shouldn't really change much about what can or cannot be done with the codebase, but since npm CLI itself is licensed under this, having this under Artistic-2.0 is going to make it easier to pull code from the CLI cleanly and hotswap the two package managers in a distant future where tink becomes the main PM.
To land this, I need signoff from everyone who's already contributed to tink. Please reply below if you're pinged on this list, otherwise I'll have to remove your contributions until you agree! I won't be accepting new PRs until this is done, either.