efegokdemir opened a new pull request, #745: URL: https://github.com/apache/maven-pmd-plugin/pull/745
## Summary When both `verbose` and `printFailingErrors` were enabled, failing violations were logged once while collecting results and again when verbose output was rendered. This made failures appear twice in the build output. `verbose` now takes precedence, while `printFailingErrors` prints only failures when verbose output is disabled. ## Changes - Centralize violation printing after failure and warning classification. - Document the precedence between `verbose` and `printFailingErrors`. - Add regression tests for both option combinations. ## Testing - `mvn -Dtest=AbstractPmdViolationCheckMojoTest test -Dspotless.apply.skip=true -Dspotless.check.skip=true -Dcheckstyle.skip=true -Drat.skip=true` — passed - `mvn test -Dspotless.check.skip=true -Dspotless.apply.skip=true -Dcheckstyle.skip=true -Drat.skip=true` — passed - `mvn checkstyle:check -DskipTests -Dspotless.check.skip=true -Dspotless.apply.skip=true -Drat.skip=true` — passed - `git diff --check` — passed Fixes #315 The contribution was prepared with AI assistance and reviewed against the repository's contribution guidelines. - [x] This pull request addresses one issue without unrelated changes. - [x] Unit tests cover the behavioral change. - [x] The contribution is licensed under the Apache License 2.0. -- 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]
