-
Notifications
You must be signed in to change notification settings - Fork 141
license: Add Synopsys MIT license terms #262
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
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.
Please update following line:
"Three licenses are in use in this repository: Apache-2.0, BSD-3-Clause and ST SLA0044."
About
Let's use "Following" instead of "Three", this will avoid to modify further if other Licenses are added |
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.
Agreed with @erwango comments, otherwise LGTM
48724e6
to
46878c4
Compare
@erwango could you review again this PR please? |
@benothmn-st It may be time to update this PR ;) |
Add Synopsys MIT license terms. Update license terms in Synopsys's headers file. Signed-off-by: Nidhal BEN OTHMEN <nidhal.benothmen@st.com>
@erwango I think we can merge now |
Add Synopsys MIT license terms