gnodet commented on PR #12402: URL: https://github.com/apache/maven/pull/12402#issuecomment-6672469522
<!-- forgebot: watermark --> The CI failure is a RAT 0.18 compatibility issue — RAT 0.18 changed how `<excludes>` patterns are interpreted, causing 3298 unapproved files to be reported, most of them under `its/`, `impl/`, and `compat/`. However, the `pom.xml` part of this PR has been superseded by commits merged directly into `master` after August 24 (specifically the Oct 30 commit "Remove redundant compiler release and plugin versions, use RAT inputExcludes", plus earlier commits excluding `.claude/` and `.worktrees/`). `master` now already uses `<inputExcludes>` with working glob patterns, which is why there's a merge conflict. What remains potentially useful from this PR: - **`its/pom.xml`** — bumping the lifecycle filter version range from `[0.11,)` to `[0.18,)` - **`NOTICE`** — http → https URL fix To move this forward, the `pom.xml` changes would need to be dropped (they're in conflict and already in master), and the PR could be narrowed to just the `its/pom.xml` and `NOTICE` changes after rebasing on current `master`. Alternatively, two small focused PRs for those two items would be the cleanest path. *This comment was created with AI assistance.* -- 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]
