Upgrade PMD to 7.0.0

[holgerfriedrich]

Add support for Java 21.

https://docs.pmd-code.org/pmd-doc-7.0.0-rc4/pmd_release_notes.html

---

Update to PMD7 gives me some headache, as AST has changed.
See migration guide, https://docs.pmd-code.org/pmd-doc-7.0.0-rc4/pmd_release_notes.html#migration-guide-for-pmd-7

Still struggling to get custom test to work properly (UseSLF4JLoggerRule.java).

@wborn Could use some help, just in case you know the details of AST parsing.....

[wborn]

I had a look and after setting a break point I could check the objects and see where all the required data is stored nowadays.
Then I could get it working again with the changes in holgerfriedrich#1.

[holgerfriedrich]

for a benchmark see #463 (comment)

[holgerfriedrich]

benchmark looks good, removed label WIP

[holgerfriedrich]

Hello @openhab/sat-maintainers , kindly asking to review/merge this PR. We need this as one of the last pieces towards a Java21 build. Thanks!

See the comparison of 0.15 and 0.16-SNAPSHOT: #463 (comment)

See comments above:

btw: Once we get #462 and #463, we should release.
The benefit it support for Java 21 (i.e. support for running SAT checks on class files generated with Java 21).

[holgerfriedrich]

@kaikreuzer When I checked a few days ago, it was still -rc4 :-)
I will quickly check if there is a functional difference and pull the final version in.

[holgerfriedrich]

A lot of changes between rc4 and final, I had to adapt a few files.
Though, now the PR is reworked and seems to work fine with the final release.

@wborn do you want to give it a try as well?

btw: The new dependency pmd-compat6 is still needed until maven-pmd-plugin gets updated.

[wborn]

It still works well for me with these changes! 👍

[kaikreuzer]

Thank you both!

[holgerfriedrich]

@kaikreuzer thanks, could you please trigger the release of 0.16?
Not sure how this works and how this gets published on the typical mvn repos....

[kaikreuzer]

@holgerfriedrich Will do. Is there any special reason why the build asks for Java 11? Or is it safe to change the constraint and build with with Java 17 instead (which seems to work well)?

[holgerfriedrich]

@holgerfriedrich Will do. Is there any special reason why the build asks for Java 11? Or is it safe to change the constraint and build with with Java 17 instead (which seems to work well)?

@kaikreuzer I typically build with newer versions on my local machine. However, the last time we tried to upgrade to 17, we have broken the release build, see #451. Something releated to the unleash plugin.

[kaikreuzer]

@holgerfriedrich Many thanks for the reminder!
I've built it with Java11 again and here we are: https://github.com/openhab/static-code-analysis/releases/tag/0.16.0

[holgerfriedrich]

@holgerfriedrich Many thanks for the reminder! I've built it with Java11 again and here we are: https://github.com/openhab/static-code-analysis/releases/tag/0.16.0

Thanks a lot, @kaikreuzer 🚀