Skip to content

[java] Update FieldDeclarationsShouldBeAtStartOfClass#3229

Merged
adangel merged 8 commits into
pmd:pmd/7.0.xfrom
oowekyala:update-FieldDeclarationsShouldBeAtStartOfClass
Apr 27, 2021
Merged

[java] Update FieldDeclarationsShouldBeAtStartOfClass#3229
adangel merged 8 commits into
pmd:pmd/7.0.xfrom
oowekyala:update-FieldDeclarationsShouldBeAtStartOfClass

Conversation

@oowekyala

Copy link
Copy Markdown
Member

Describe the PR

Update the rule FieldDeclarationsShouldBeAtStartOfClass as is.

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 423 violations,
introduces 678 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 9259 violations,
introduces 9040 new violations, 1466 new errors and 0 new configuration errors,
removes 49331 violations, 25 errors and 2 configuration errors.
Full report
Compared to pmd/7.0.x:
This changeset changes 0 violations,
introduces 678 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 9231 violations,
introduces 8361 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 self-assigned this Apr 27, 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, I'll merge that now.

Thanks for investigating and fixing the build btw (0ea8dc4)

@adangel adangel merged commit 25463d9 into pmd:pmd/7.0.x Apr 27, 2021
@oowekyala oowekyala deleted the update-FieldDeclarationsShouldBeAtStartOfClass branch April 27, 2021 18:19
@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