Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set appropriate log level in pmd-maven-plugin #5032

Closed
delanym opened this issue May 23, 2024 · 1 comment
Closed

Set appropriate log level in pmd-maven-plugin #5032

delanym opened this issue May 23, 2024 · 1 comment
Labels
a:bug PMD crashes or fails to analyse a file.

Comments

@delanym
Copy link

delanym commented May 23, 2024

The org.apache.maven.plugins.pmd.PmdViolationCheckMojo goal logs violations at "info" level.
See example at #4001

I filter Maven log output based on the log level, so I miss these after setting printFailingErrors to true.

If the failOnViolation is true, I expect violations logged at error level.
If the failOnViolation is false, I expect violations logged at warning level.

@delanym delanym added the a:bug PMD crashes or fails to analyse a file. label May 23, 2024
@jsotuyod
Copy link
Member

This repository is about PMD itself. Maven PMD Plugin lives at apache.org -> https://issues.apache.org/jira/projects/MPMD/summary

As for pmd itself, the failOnViolation flag (or equivalent, depending on whether this is a cli, ant or programmatic invocation) should not affect where errors are reported. Failure is determined by the output (build status, status code, etc) and not the channel.

@jsotuyod jsotuyod closed this as not planned Won't fix, can't repro, duplicate, stale May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:bug PMD crashes or fails to analyse a file.
Projects
None yet
Development

No branches or pull requests

2 participants