Skip to content

[all] Use slf4j (Part 2)#3794

Closed
adangel wants to merge 20 commits into
pmd:pmd/7.0.xfrom
adangel:pmd7-896-slf4j-migrations
Closed

[all] Use slf4j (Part 2)#3794
adangel wants to merge 20 commits into
pmd:pmd/7.0.xfrom
adangel:pmd7-896-slf4j-migrations

Conversation

@adangel

@adangel adangel commented Feb 18, 2022

Copy link
Copy Markdown
Member

Describe the PR

Support "--debug" flag for slf4j-simple
This is a hacky way: It reconfigures SimpleLogger to use
a special print stream. What ever is printed there is
forwarded to ant's project logger.

Removed the deprecated and now unused classes in
net.sourceforge.pmd.util.log.
This is at least needed for apex jorje, but might be handy
for any other library that logs through jul.
Need to remove all handlers first, as java by default
adds a ConsoleLogger. The SLF4JBridgeHandler should be
the only handler registered.
@adangel adangel added this to the 7.0.0 milestone Feb 18, 2022
@adangel adangel marked this pull request as draft February 18, 2022 17:23
@adangel adangel mentioned this pull request Feb 18, 2022
9 tasks
@ghost

ghost commented Feb 18, 2022

Copy link
Copy Markdown
2 Messages
📖 Compared to pmd/7.0.x:
This changeset changes 0 violations,
introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 0 errors and 0 configuration errors.
Full report
📖 Compared to master:
This changeset changes 49302 violations,
introduces 26828 new violations, 5 new errors and 0 new configuration errors,
removes 146082 violations, 25 errors and 3 configuration errors.
Full report
Compared to pmd/7.0.x:
This changeset changes 0 violations,
introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 0 errors and 0 configuration errors.
Full report
Compared to master:
This changeset changes 49302 violations,
introduces 26828 new violations, 5 new errors and 0 new configuration errors,
removes 146082 violations, 25 errors and 3 configuration errors.
Full report
Compared to pmd/7.0.x:
This changeset changes 0 violations,
introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 0 errors and 0 configuration errors.
Full report
Compared to master:
This changeset changes 49302 violations,
introduces 26828 new violations, 5 new errors and 0 new configuration errors,
removes 146082 violations, 25 errors and 3 configuration errors.
Full report

Generated by 🚫 Danger

@adangel adangel marked this pull request as ready for review February 18, 2022 18:46
@adangel

adangel commented Feb 19, 2022

Copy link
Copy Markdown
Member Author

Closed in favor of #3789

@adangel adangel closed this Feb 19, 2022
@adangel adangel deleted the pmd7-896-slf4j-migrations branch February 26, 2022 10:29
@adangel adangel removed this from the 7.0.0 milestone Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant