fix #18 - Add GitHub Action to enforce Apache 2.0 license headers#27
Conversation
fantonangeli
requested review from
Copilot,
lornakelly and
ricardozanini
and removed request for
Copilot
|
@fantonangeli you must sign your commits with a key that matches your email. |
Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
fantonangeli
force-pushed
the
issue-18-Add-GitHub-Action-to-enforce-Apache-20-license-headers-2
branch
from
5dd02f0 to
40a08ea
Compare
There was a problem hiding this comment.
Pull request overview
Adds automated license-header enforcement using Apache RAT so pull requests fail if files without approved licensing are introduced, supporting the repository’s Apache 2.0 compliance goal.
Changes:
- Add a CI workflow that runs Apache RAT on pushes to
mainand on PR events. - Introduce
.rat-excludesto exclude certain non-source/config/generated files from RAT scanning. - Add Apache 2.0 license headers to
README.mdand.gitignore.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| README.md | Adds an Apache 2.0 license header in an HTML comment. |
| .rat-excludes | Defines files to exclude from Apache RAT scanning. |
| .gitignore | Adds an Apache 2.0 license header comment block. |
| .github/workflows/ci_check_license_headers.yaml | New workflow that downloads and runs Apache RAT, failing on unapproved files. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
fantonangeli
force-pushed
the
issue-18-Add-GitHub-Action-to-enforce-Apache-20-license-headers-2
branch
from
2947dc9 to
68eca73
Compare
Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
…hanges#r2897289283 Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
…hanges#r2897676663 Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
|
@lornakelly are you going to review this PR? |
…-Action-to-enforce-Apache-20-license-headers-2 Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
Yes, apologies, I didnt know it needed 2 reviews, im used to one person grabbing a PR to review on first come first served basis and saw you had started review before me. I understand now and will make sure to review all future PRs as well even if someone is already reviewing |
fantonangeli
deleted the
issue-18-Add-GitHub-Action-to-enforce-Apache-20-license-headers-2
branch
Closes #18
Summary
Add a GitHub Actions workflow that validates Apache 2.0 license headers are present in all applicable source files and fails the build if headers are missing. The check must execute on every Pull Request.
Goals
Ensure all source files comply with Apache 2.0 licensing requirements.
Non-Goals