slachiewicz commented on PR #681: URL: https://github.com/apache/maven-checkstyle-plugin/pull/681#issuecomment-5226348495
Added a second commit removing the unused `maven-compat` test dependency, and dropping it from the dependency-analysis `<ignoredDependencies>` list at the same time. Nothing in the plugin imports anything from maven-compat — the declaration had simply outlived whatever needed it, and the ignore rule was what kept the analysis quiet about it. Tests: 26, 0 failures, 1 skipped — unchanged by either commit. Kept on this branch rather than opened as a separate PR since it touches adjacent lines in the same file. Happy to split it out if you would rather review them apart. This is part of a wider survey of which projects still declare maven-compat versus which genuinely use it; several turn out never to touch it, while others depend on it in ways that are not trivially removable. -- 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]
