Skip to content

[java] Update rule OnlyOneReturn#3227

Merged
adangel merged 3 commits into
pmd:pmd/7.0.xfrom
oowekyala:update-OnlyOneReturn
Apr 23, 2021
Merged

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

Conversation

@oowekyala

Copy link
Copy Markdown
Member

Describe the PR

Related issues

Ready?

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

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

ghost commented Apr 19, 2021

Copy link
Copy Markdown
2 Messages
📖 Compared to pmd/7.0.x:
This changeset changes 0 violations,
introduces 6056 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 9217 violations,
introduces 8403 new violations, 1 new errors and 0 new configuration errors,
removes 15871 violations, 10 errors and 2 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 9217 violations,
introduces 8340 new violations, 1 new errors and 0 new configuration errors,
removes 15871 violations, 10 errors and 2 configuration errors.
Full report

Generated by 🚫 Danger

@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.

👍
It seems a couple of false negatives are fixed now

@adangel adangel self-assigned this Apr 23, 2021
@adangel adangel merged commit 6c9eee8 into pmd:pmd/7.0.x Apr 23, 2021
@oowekyala oowekyala deleted the update-OnlyOneReturn branch April 23, 2021 13:26
@adangel adangel mentioned this pull request Jan 23, 2023
55 tasks
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