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

Change to the Apache 2 licence #53

Merged
merged 1 commit into from
Oct 6, 2020
Merged

Change to the Apache 2 licence #53

merged 1 commit into from
Oct 6, 2020

Conversation

thomasbird
Copy link
Member

It is proposed to change the licence of this project from MIT to Apache 2.0.

To quote from exygy.com who compare the two:

The Apache license has a similar philosophy to the MIT, but uses more words. The wordiness creates greater specificity about contributors’ obligations, which might help in a dispute. But it also can be a turnoff — “Do I need to have my lawyer look at this?” comes up more with Apache than MIT. It works well for organizations or projects that are larger and managing more contributors, but don’t care about others commercializing the work. It also can help bring on board organizations that are more concerned about software patents or patent trolls.

I think the main gist is that it is very similar to the MIT licence, but lays the foundation to help deal with some situations that could crop up especially around patients.

What do people think about this proposed change?

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 95.968% when pulling 01f37ed on licence/apache into c1050ce on master.

@deanmalmgren
Copy link
Collaborator

This sounds reasonable to me. We first licensed this under MIT for simplicity, but if you have good reasons to have a different open source license that supports the use cases of this package, I'm supportive 👍 👍

@thomasbird thomasbird merged commit e1b2343 into master Oct 6, 2020
@thomasbird thomasbird deleted the licence/apache branch November 4, 2020 10:58
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.

None yet

3 participants