-
Notifications
You must be signed in to change notification settings - Fork 8.2k
scripts: ci: check_compliance: Warn on missing Apache-2.0 license #99003
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
base: main
Are you sure you want to change the base?
Conversation
| paths, | ||
| "License may not be allowed", | ||
| "warning", | ||
| "error", |
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.
this seems like an unrelated change? We'd first need to add all okay-ish license to /LICENSES until we can turn this into an error?
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.
Yeah, the PR was a draft for that reason, the intention is to add the allowed licenses alongside that change.
But I wanted to get some feedback first 🙂
EDIT: License files added
f9e97ce to
7f9c2a6
Compare
7f9c2a6 to
1e40022
Compare
Print a compliance warning if a modified file is not licensed under Apache-2.0. Signed-off-by: Pieter De Gendt <pieter.degendt@basalte.be>
Add license files for the reuse tool, downloaded using reuse download. Signed-off-by: Pieter De Gendt <pieter.degendt@basalte.be>
Increase the severity if a file is licensed without it being allowed. Signed-off-by: Pieter De Gendt <pieter.degendt@basalte.be>
1e40022 to
5f7a0e3
Compare
|



Print a compliance warning if a modified file is not licensed under Apache-2.0.
Missing license files are now turned into compliance errors.