elharo opened a new pull request, #556: URL: https://github.com/apache/maven-jar-plugin/pull/556
## Summary `mvn dependency:analyze` reports 4 unused declared dependencies. Three are genuinely unused and can be removed. The fourth (guice) is a false positive required by the test harness. ## Changes **Removed (3):** - `maven-api-xml:provided` — no code references; already transitive via `maven-api-core` - `maven-xml:provided` — no code references; already transitive via `maven-testing` - `maven-core:test` — no code references; already transitive via `maven-testing` **Kept with suppression (1):** - `guice:test` — false positive. Required at test runtime by Eclipse Sisu (used by maven-testing harness for DI). Test-scope transitive deps at provided scope are excluded per Maven's scope table, so guice must be declared explicitly. Added to `<ignoredUnusedDeclaredDependencies>`. ## Verification - No imports from any of the 4 packages exist in the project source code - Each removed dependency is already available transitively through other declared dependencies - guice suppression is scoped to the `:classes` classifier -- 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]
