-
Notifications
You must be signed in to change notification settings - Fork 39
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 credits #219
update credits #219
Conversation
README.md
Outdated
@@ -88,3 +88,18 @@ For example, to release version 0.1.2 you need to pull devel, checkout master, p | |||
## Contributing | |||
Please see [CONTRIBUTING](CONTRIBUTING.md). | |||
|
|||
### Credits | |||
|
|||
First-party code is copyright Optimizely, Inc. and contributors, licensed under Apache 2.0. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you include the location of the first party code here so we can be explicitt about it
README.md
Outdated
|
||
This software incorporates code from the following open source projects: | ||
|
||
**Otimizely.ab:core-api** [https://github.com/optimizely/java-sdk](https://github.com/optimizely/java-sdk) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: typo "Otimizely"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Can this be merged? |
No description provided.