-
Notifications
You must be signed in to change notification settings - Fork 52
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
chore: add license to module #554
Conversation
f1247f1
to
354f230
Compare
Signed-off-by: Todd Baert <[email protected]>
354f230
to
671aba8
Compare
same "printed page" as the copyright notice for easier | ||
identification within third-party archives. | ||
|
||
Copyright [yyyy] [name of copyright owner] |
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.
Seems like more pain to maintain, but I suppose technically these should be updated with the open feature details, right? I see that this isn't the case in the root LICENSE file though so perhaps we can just ignore it.
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.
oof I missed this before the merge. I will udpate.
Our license tooling recently started failing (due to updates) because our modules are missing licenses (we only have them at the root). This adds licenses to all modules.