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
Upgrade maven-pmd-plugin to 3.22.0 which uses PMD 7 by default #7577
base: main
Are you sure you want to change the base?
Conversation
…EmptyIfStmt, EmptyInitializer, EmptyStatementBlock, EmptySwitchStatements, EmptySynchronizedBlock, EmptyTryBlock, and EmptyWhileStmt
f5d5d32
to
3036b99
Compare
Because I don't like scrolling
Nice one! Ambitious, too 🤣 |
…llections, fix some try-with-resources
…nly, must not be used in configuration
This turns out to be much more work than anticipated, unfortunately I've no run out of time due to upcoming holiday, so this PR will stall from my side for a couple of weeks. Anyone willing to spend some time on it is quite welcome 🤣 |
Migrated a couple of modules while I was travelling. |
And a few more under "web" (but not all yet) |
This upgrades to the current release of the maven-pmd-plugin which uses PMD 7.0.0 by default.
While at it also update the build steps in the GH workflow
In 7.0.0 some rules have changed, this requires some changes in the selected rules.
As a result some PMD warnings are now triggered that need fixing.
Checklist
main
branch (backports managed later; ignore for branch specific issues).For core and extension modules:
[GEOS-XYZWV] Title of the Jira ticket
.