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

Update license to Apache License Version 2.0 #19

Merged
merged 16 commits into from
Aug 26, 2024
Merged

Update license to Apache License Version 2.0 #19

merged 16 commits into from
Aug 26, 2024

Conversation

qingyun-wu
Copy link

@qingyun-wu qingyun-wu commented Aug 26, 2024

Why are these changes needed?

Summary

This pull request changes the project license from MIT to Apache 2.0 for all new and modified code, while preserving the MIT license for original code.

Changes Made

  • Added new LICENSE file with Apache 2.0 license text
  • Renamed original MIT license to LICENSE_original_MIT and included in the folder license_original
  • Added NOTICE file explaining the dual licensing
  • Updated README.md to reflect license changes
  • Updated file headers in modified files to Apache 2.0
  • Updated the contributing guide to give contributors instructions

Rationale

The Apache 2.0 license was chosen for the following reasons:

  • It provides explicit patent rights, which offers better protection for contributors and users
  • It's compatible with many other open-source licenses, including MIT
  • It's widely recognized and trusted in the open-source community

Compliance

  • Original MIT-licensed code remains under MIT license (the original license is included in the license_original/LICENSE_original_MIT file)
  • All new and modified code will be under Apache 2.0
  • Dual licensing is clearly explained in README and NOTICE files

Next Steps

  • Review and merge this PR
  • Update project metadata on GitHub to reflect the license change
  • Notify existing contributors and users about the license change

Related issue number

Checks

README.md Outdated Show resolved Hide resolved
NOTICE.md Show resolved Hide resolved
Co-authored-by: Chi Wang <[email protected]>
Copy link
Collaborator

@Hk669 Hk669 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me. thanks for the efforts in updating the license, really appreciated @qingyun-wu

@sonichi sonichi merged commit 47cbb3a into main Aug 26, 2024
128 of 142 checks passed
@sonichi sonichi deleted the license branch August 26, 2024 21:05
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.

4 participants