Skip to content

Upgrade PMD to 7.14.0 and Checkstyle to 10.25.0 #484

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

Merged
merged 2 commits into from
Jun 9, 2025

Conversation

holgerfriedrich
Copy link
Member

@holgerfriedrich holgerfriedrich commented Jun 8, 2025

Separate PRs, do not squash!

Comparison of output before/after the patch will be made available once my build finishes.

Upgrades PMD from 7.13.0 to 7.14.0.
Now defaults to parallel execution.

For release notes, see:
https://github.com/pmd/pmd/releases/tag/pmd_releases%2F7.14.0

Signed-off-by: Holger Friedrich <[email protected]>
@holgerfriedrich holgerfriedrich requested a review from a team as a code owner June 8, 2025 16:38
Upgrades Checkstyle from 10.23.0 to 10.25.0.

For release notes, see:
https://github.com/checkstyle/checkstyle/releases

Signed-off-by: Holger Friedrich <[email protected]>
@holgerfriedrich
Copy link
Member Author

Still working as expected, missing SimplifyBooleanReturns is expected, as we removed this after release 0.17.
grafik

@kaikreuzer kaikreuzer merged commit 455017c into openhab:main Jun 9, 2025
2 checks passed
@kaikreuzer kaikreuzer added this to the 0.18.0 milestone Jun 9, 2025
@holgerfriedrich
Copy link
Member Author

@kaikreuzer can you please create a release?

@holgerfriedrich holgerfriedrich deleted the pr-all branch June 9, 2025 09:01
@kaikreuzer
Copy link
Member

Is it urgent?

@holgerfriedrich
Copy link
Member Author

@kaikreuzer

Is it urgent?

Not really, as the functionality is largely the same. But it would be appreciated. :-)
At the moment, we have some clutter in the report, which will be gone after an upgrade (refers to SimplifyBooleanReturns we removed in an earlier commit).

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