Uh oh!
There was an error while loading. Please reload this page.
[MINOR][DOCS] Add license header at docs/_plugins - #44647
Conversation
nchammas
commented
Jan 15, 2024
Should we update the RAT excludes? Line 20 in c0ff0f5 I believe this is why the build did not catch these missing licenses. |
HyukjinKwon
commented
Jan 15, 2024
I think we should probably update. |
nchammas
commented
Jan 15, 2024
FYI - I am checking upstream with the RAT devs to see if we can improve how the tool works. Having RAT scan Because we use the CLI and not the Maven plugin (some history here), RAT does not automatically look at our I am following RAT-323 and RAT-352 to see if RAT will get some usability improvements for the CLI in the near future. |
What changes were proposed in this pull request?
This PR adds license header to
docs/_pluginsfiles.Why are the changes needed?
To comply Apache License 2.0
Does this PR introduce any user-facing change?
No, dev-only.
How was this patch tested?
Existing CI should verify it e.g., linter.
Was this patch authored or co-authored using generative AI tooling?
No.