Copilot commented on code in PR #11801:
URL: https://github.com/apache/maven/pull/11801#discussion_r2935942042
##########
pom.xml:
##########
@@ -825,6 +825,16 @@ under the License.
! Excluded the license files itself cause they do not have a
license of themselves.
-->
<exclude>src/main/appended-resources/licenses/**</exclude>
+ <!--
+ ! Exclude Eclipse IDE project files (not cleaned by mvn clean)
+ -->
+ <exclude>**/.classpath</exclude>
+ <exclude>**/.project</exclude>
+ <exclude>**/.settings/**</exclude>
+ <!--
+ ! Exclude build output files that may exist in stale or
non-module directories
+ -->
+ <exclude>**/.plxarc</exclude>
Review Comment:
The RAT failure in the linked issue report includes many generated files
under `target/` (e.g., `target/site/**`, `target/checkstyle-*`, etc.). This
change only excludes IDE metadata and `.plxarc`, so RAT will still report
unapproved licenses when `target/` exists (e.g., after running a build without
`mvn clean`). Consider excluding `**/target/**` (or enabling RAT default
excludes if they’re currently disabled) to make the fix complete and resilient
to stale build output.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
