gnodet opened a new pull request, #12613: URL: https://github.com/apache/maven/pull/12613
## Summary Add missing Apache License headers to files flagged by ATR (Apache Trusted Releases) during the rc-6 release review. - **Java test files**: `MyMojoTest.java`, `App.java`, `AppTest.java` — added standard ASF license header - **Shell script**: `prepare.sh` — added `#`-style license header after shebang - **Config files**: `Jenkinsfile`, `.asf.yaml` — added license headers - **Service loader files**: `javax.annotation.processing.Processor`, `org.apache.maven.api.di.Inject` — added `#`-comment license headers (valid per SPI spec) - **Test XML files**: 77 `.pom`, `pom.xml`, and `maven-metadata.xml` files in `compat/maven-compat/src/test/resources/inheritance-repo/` and `src/test/remote-repo/` — added XML comment license headers - **Placeholder file**: `placeholder.txt` — added plain text license header ### Remaining ATR concerns (cannot be fixed): - **Binary `.jar` files** (~50 files in test resources) — cannot add text headers to binary files - **Third-party license texts** (`EPL-1.0.txt`, `CDDL+GPLv2-with-classpath-exception.txt`, etc.) — cannot add Apache header to third-party licenses - **`Bad.java`** — intentionally invalid Java file for MNG-5208 test - **`source.txt`** — test asserts exact `"Hello World!"` on first line These remaining items are already excluded in the Maven Rat plugin configuration and are standard false positives. ## Test plan - [ ] Verify `mvn verify` still passes (XML comment headers are ignored by Maven's POM parser) - [ ] Verify inheritance tests in `compat/maven-compat` still pass - [ ] Verify ATR re-scan shows fewer concerns 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
