-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
[all] Use slf4j #896
Comments
@adangel I totally support this, I think it's the most versatile approach. Also, the use of placeholders ( We may have trouble with things such as Jorje internally using java util logging, although we are currently silencing some of it's output, so it may not be an issue. However, this may imply changes on how tools integrate with PMD, so maybe this should be part of 7.0.0? |
Yes, probably something for a major version change. Also the command line output (format) might change. By the way - it's no problem to integrate j.u.logging into slf4j. I did the same for the latest m-pmd-p, where I redirect PMD's logging into maven (https://github.com/apache/maven-pmd-plugin/blob/ffa915f02bc70a9eaea57a24093169e589e8e8be/src/main/java/org/apache/maven/plugins/pmd/AbstractPmdReport.java#L547) |
@adangel / @oowekyala any reason why this task is still open? I don't see any pending task in the description. |
This is done for PMD7 via #3789 |
Part of #2524
Rationale: In order to make it easier to integrate PMD into other tools, it might be better to use the de-facto-standard slf4j.
We could then provide some basic simplelogger configuration in our binary distribution, but integrations have full freedom on which logging framework they want to use.
This would at slf4j-api as an additional compile-time dependency and maybe slf4j-impl as runtime dependency in our pmd-bin-xxx.zip distribution.
Currently we are using java util logging.
Tasks:
configuration
-debug
)The text was updated successfully, but these errors were encountered: