Skip to content

Update licensing #5

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 18, 2016
Merged

Update licensing #5

merged 2 commits into from
Sep 18, 2016

Conversation

kgryte
Copy link
Member

@kgryte kgryte commented Sep 16, 2016

Updates the license from MIT to Apache-2.0, where the Apache-2.0 includes additional provisions related to patent termination and indemnification.

@kgryte kgryte added Needs Review A pull request which needs code review. license labels Sep 16, 2016
@kgryte
Copy link
Member Author

kgryte commented Sep 16, 2016

Note: the license field in package.json fields is intentionally left blank. Similar to other fields, like dependencies, version, and name, this can be auto-generated upon decomposition. The only time this field SHOULD have a value is when the package requires special licensing, such as a joint license; e.g., Apache-2.0 AND BSL-1.0. In this case, a license value implies overriding the default project license.

@codecov-io
Copy link

Current coverage is 86.2200%

Sunburst

No tracked files adjusted.

No coverage report found for develop at f972154.

Powered by Codecov. Last update f972154...7e1f785

@kgryte
Copy link
Member Author

kgryte commented Sep 16, 2016

@Planeshifter Merged develop, so should be good for review.

@Planeshifter
Copy link
Member

LGTM.

@kgryte kgryte merged commit a05ecb8 into develop Sep 18, 2016
@kgryte kgryte deleted the feature/licensing branch September 20, 2016 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Planeshifter Planeshifter

Labels
Needs Review A pull request which needs code review.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kgryte @codecov-io @Planeshifter