Skip to content
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

[doc] Document rule changes for PMD 7 #4501

Merged
merged 22 commits into from
Apr 29, 2023
Merged

Conversation

@adangel adangel added this to the 7.0.0 milestone Apr 23, 2023
@pmd-test
Copy link

pmd-test commented Apr 23, 2023

1 Message
📖 Compared to master:
This changeset changes 0 violations,
introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 13 errors and 0 configuration errors.
Full report
Compared to master:
This changeset changes 0 violations,
introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 13 errors and 0 configuration errors.
Full report
Compared to master:
This changeset changes 0 violations,
introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 13 errors and 0 configuration errors.
Full report
Compared to master:
This changeset changes 0 violations,
introduces 0 new violations, 0 new errors and 0 new configuration errors,
removes 0 violations, 13 errors and 0 configuration errors.
Full report
No regression tested rules have been changed.

Generated by 🚫 Danger

@adangel adangel mentioned this pull request Apr 24, 2023
55 tasks
and not anymore on formal parameter.

Refs pmd#3127
Refs pmd#3123
…mum"

While we calculate the ratio between number of statements and number of labels,
we shouldn't pretend that this is an exact rule - it's anyway a best guess,
that high switch density is a problem. I don't think, it makes a big difference
between 10.2 and 10. You e.g. want to report, when the density is bigger than 10
or bigger than 5. We don't need the report level accuracy here.

Refs pmd#3123
@adangel adangel marked this pull request as ready for review April 28, 2023 07:34
@adangel
Copy link
Member Author

adangel commented Apr 28, 2023

So, that's finally ready now.

Most important points:

@adangel adangel merged commit e493db6 into pmd:master Apr 29, 2023
3 checks passed
adangel added a commit to adangel/pmd that referenced this pull request Apr 29, 2023
@adangel adangel deleted the doc-pmd7-rule-changes branch April 29, 2023 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment