Skip to content

[java] Update rule InvalidLogMessageFormat#3512

Merged
adangel merged 3 commits into
pmd:pmd/7.0.xfrom
oowekyala:update-InvalidLogMessageFormat
Sep 30, 2021
Merged

[java] Update rule InvalidLogMessageFormat#3512
adangel merged 3 commits into
pmd:pmd/7.0.xfrom
oowekyala:update-InvalidLogMessageFormat

Conversation

@oowekyala

Copy link
Copy Markdown
Member

Describe the PR

Ready?

  • Added unit tests for fixed bug/feature
  • Passing all unit tests
  • Complete build ./mvnw clean verify passes (checked automatically by github actions)
  • Added (in-code) documentation (if needed)

@oowekyala oowekyala added this to the 7.0.0 milestone Sep 19, 2021
@ghost

ghost commented Sep 19, 2021

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 31204 violations,
introduces 21203 new violations, 1 new errors and 0 new configuration errors,
removes 137237 violations, 8 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 31204 violations,
introduces 21203 new violations, 1 new errors and 0 new configuration errors,
removes 137237 violations, 8 errors and 3 configuration errors.
Full report

Generated by 🚫 Danger

@oowekyala oowekyala changed the title Update rule InvalidLogMessageFormat [java] Update rule InvalidLogMessageFormat Sep 21, 2021
@adangel adangel self-assigned this Sep 30, 2021

@adangel adangel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.
Interestingly, we don't have any matches of this rule in our regression tester projects...

@adangel adangel merged commit 4b112e8 into pmd:pmd/7.0.x Sep 30, 2021
@adangel adangel mentioned this pull request Jan 23, 2023
55 tasks
@oowekyala oowekyala deleted the update-InvalidLogMessageFormat branch May 13, 2024 21:53
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.

2 participants