slachiewicz opened a new pull request, #684: URL: https://github.com/apache/maven-checkstyle-plugin/pull/684
The header-override variant of the MCHECKSTYLE-357 integration test pins `com.puppycrawl.tools:checkstyle` 8.15 — the version that was current when the test was added in 2018. Every other IT that needs an explicit Checkstyle dependency filters in `@checkstyleVersion@` (currently 9.3), so this one now does too. That silences both open Dependabot alerts on this repo ([#2](https://github.com/apache/maven-checkstyle-plugin/security/dependabot/2), `< 8.18`, and [#3](https://github.com/apache/maven-checkstyle-plugin/security/dependabot/3), `< 8.29`) — same manifest, same dependency. Two rules in the inline ruleset had to move with the upgrade, since the IT exercises `checkstyleRules`/`checkstyleRulesHeader` rather than any particular rule: - `MissingDeprecated` lost its `skipNoJavadoc` property in Checkstyle 8.24. - `LineLength` is a child of `Checker`, not `TreeWalker`, since 8.24. Verified locally: `mvn -Prun-its -Dinvoker.test=MCHECKSTYLE-357-with-header-override verify` passes, the run resolves `checkstyle-9.3.jar`, and the generated `target/checkstyle-rules.xml` still carries the overridden DOCTYPE header the test is about. Generated-by: Claude Opus 5 (1M context) -- 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]
