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
Submit related projects in NOTICE. #683
Conversation
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.
small addition required: dependencies should be listed with the license they are distributed with.
NOTICE.txt
Outdated
mockito: http://site.mockito.org/ | ||
QOS.ch (slf4j): https://www.slf4j.org/ | ||
powermock: https://github.com/powermock/powermock | ||
Eclipse (Jetty): https://www.eclipse.org/jetty/ |
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.
for each of the dependencies you need to say which license they use.
Add licenses for each dependency.
@michaelsembwever The season I didn't add license, is because the two examples don't include that, https://github.com/apache/cassandra/blob/trunk/NOTICE.txt and https://github.com/apache/tiles/blob/trunk/NOTICE.txt . But anyway, I have added them already. :) |
looks good! thanks @wu-sheng |
@michaelsembwever :
I add the NOTICE file, which includes the related projects in distribution. You can find many dependencies in our project is
provided
only, so I did't list them all, we just instrument them, not using them.FYI @hanahmily you need to create the same NOTICE file in
incubator-skywalking-ui
repository too.CC @apache/skywalking-committers