-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add .rat-excludes file to each repository #1613
Add .rat-excludes file to each repository #1613
Comments
Thank you @tkobayas, I have something similar in my head longer time. What I see as we should do as part of this ticket is to unify the approach. Because there were already some initiatives to have licenses checks. apache-rat-pluginUsed for example in kogito-runtimes https://github.com/apache/incubator-kie-kogito-runtimes/blob/main/pom.xml
checkstyle.header.templateUsed in a lot of our maven projects
I think we should avoid the situation we have in parallel use of:
I think we should use just one of the above, while the last seems to me as best candidate as it is not bind to maven projects. What is your opinion? |
Hi @jomarko
Yeah, +1 for Indeed, I merged apache/incubator-kie-drools#6149 with removing Also thanks for mentioning that |
Also a +1 from me about using the CLI tool. |
This a partial fix for the issue apache/incubator-kie-issues#1613 as we open one PR per repository.
This is partial fix of the issue apache/incubator-kie-issues#1613 as we open one PR per repository.
This a partial fix for the issue apache/incubator-kie-issues#1613 as we open one PR per repository.
FYI, "kogito-images" will be not updated with "CI :: Check license headers" workflow, see the explanation here apache/incubator-kie-kogito-images#1811 (comment). |
In order to check
All files have license headers if necessary.
https://kie.apache.org/docs/community/verify/ , add.rat-excludes
to the root directory of each repository.CI :: Check license headers
GitHub Actions workflow incubator-kie-optaplanner#3130 and kie-issues#1613: MigrateCI :: Check license headers
apache-rat v0.16.1 incubator-kie-optaplanner#3144CI :: License headers
GitHub Actions workflow incubator-kie-kogito-runtimes#3714CI :: Check license headers
GitHub Actions workflow incubator-kie-kogito-apps#2111incubator-kie-kogito-images [10.0.x] kie-issues#1613: AddCI :: Check license headers
GitHub Actions workflow incubator-kie-kogito-images#1811CI :: Check license headers
GitHub Actions workflow incubator-kie-tools#2746For example, running
can easily confirm license headers.
Each repository committers should add ignorable file names/paths in
.rat-excludes
.Note:
.*\.csv
. We cannot express file path.--input-exclude-file
option. e.g.**/optaplanner-examples/data/**/import/**
The text was updated successfully, but these errors were encountered: