Forward port: Add Apache license headers to fix ATR concerns - #12615
Conversation
Add missing Apache License headers to files flagged by ATR: - Java test files (MyMojoTest.java, App.java, AppTest.java) - Shell script (prepare.sh) - Config files (Jenkinsfile, .asf.yaml) - Service loader files (Processor, di.Inject) - Test pom/metadata XML files in inheritance-repo - Test placeholder files Binary .jar files and third-party license texts (EPL, CDDL, BSD) remain without headers as they cannot be modified. Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
gnodet
left a comment
gnodet
left a comment
There was a problem hiding this comment.
Clean forward port of #12613 (already merged on maven-4.0.x). Mechanical addition of standard Apache License 2.0 headers to 87 files flagged by ATR checks. All headers use the correct comment syntax per file type (/* */ for Java/Groovy, <!-- --> for XML/POM, # for YAML/SPI/shell, plain text for .txt).
Three minor incidental changes (all benign):
- Jenkinsfile —
#!groovyshebang removed (optional, does not affect pipeline execution) - NOTICE — URL modernized from
httptohttps - AppTest.java — Import ordering adjusted (formatter-induced cleanup)
Shell script correctly preserves #!/bin/sh shebang on line 1 with license after. XML files correctly preserve <?xml?> declarations.
(Posting as COMMENT because GitHub does not allow self-approval — this would otherwise be an APPROVE.)
This review was generated by an AI agent and may contain inaccuracies. Please verify all suggestions before applying.
Claude Code on behalf of gnodet
|
@gnodet Please assign appropriate label to PR according to the type of change. |
1 participant
Summary
maven-4.0.xtomaster.asf.yaml,Jenkinsfile, and placeholder text files🤖 Generated with Claude Code