Skip to content
This repository was archived by the owner on Mar 3, 2023. It is now read-only.

Add license#1155

Merged
kevinsawicki merged 1 commit intomasterfrom
ks-license
Nov 28, 2013
Merged

Add license#1155
kevinsawicki merged 1 commit intomasterfrom
ks-license

Conversation

@kevinsawicki
Copy link
Contributor

We should probably decide on a license before the private alpha starts.

This PR is an attempt at that with an initial choice of the MIT license.

Once we agree on one for atom/atom, I'll update all other package repos with the same license.

/cc @hoolio @defunkt

@mcolyer mcolyer mentioned this pull request Nov 22, 2013
48 tasks
@defunkt
Copy link
Contributor

defunkt commented Nov 23, 2013

I've always been a fan of MIT. I don't know what we should be thinking about legally here, but it's my go-to license and I love seeing GitHub open source software licensed under it.

@hoolio
Copy link

hoolio commented Nov 26, 2013

Everyone likes MIT b/c it's short and sweet, but personally I think that Apache will work better for us. MIT is silent on patent issues, for instance, and Apache makes clear that third-parties have patent rights over the work as long as they don't go around and sue someone on a silly patent claim.

@stevekrause and @talniv -- thoughts?

@kevinsawicki
Copy link
Contributor Author

@hoolio Just a heads up, @stevekrause and @talniv were not in the Atom org. at the time of your comment and since this repo is currently private they wouldn't get that @mention ping.

I added them to the org. so they should be pinged by this comment.

@nathansobo
Copy link
Contributor

I'm 👍 on anything that can make the world a better place with regard to silly patent claims.

@talniv
Copy link

talniv commented Nov 26, 2013

I'm all for Apache for the reason @hoolio mentioned.
Thanks @kevinsawicki.

@kevinsawicki
Copy link
Contributor Author

I switched it to Apache, anyone else have any other feedback before I merge this in?

@stevekrause
Copy link

Works for me. Thanks, @kevinsawicki.

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.

6 participants