-
Notifications
You must be signed in to change notification settings - Fork 865
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
Add NOTICE and full copy of Apache 2.0 license text #2318
Conversation
RE: This ticket and #2312 -- Why are we doing this? What are we trying to fix? |
This is how it is done according to https://www.apache.org/foundation/license-faq.html#Apply-My-Software
|
If we use the Apache license we should do it how they suggest -> approved |
@jackiekazil can I merge the PR or do you have more reservations? |
Thank you for checking! I am good. Please merge |
Supersedes #2312.