-
Notifications
You must be signed in to change notification settings - Fork 21
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
Read also NOTICE file from the repository, as used/required by the Apache 2.0 license #1683
Comments
Hi @mitar, Thanks for your report! It's definitely doable, thanks I didn't know the Apache 2.0 licence required the NOTICE file to be bundled as well, would you have a link I can read? It seems quite easy to do, the only question I have is if it should be an option, or not. Said differently:
|
From the license:
|
Hi @mitar, I just published version 3.3.0 with this feature (the notice file will be included in the third party output). |
Awesome, thank you! |
First, thank you for this great plugin. It really nicely solves a big paint point of dealing with licenses.
Apache 2.0 license requires to bundle also any NOTICE file, if one exists in the repository, not just the LICENSE file. Could you please read that file as well and append it to
licenseText
, if it exists? Currently the code reads onlyLICENSE*
file?The text was updated successfully, but these errors were encountered: