Use Convention Over Configuration (CoC) Instead of Explicit PMD Rules #8

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.

Sign up for GitHub

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

Jump to bottom

Open

Pankraz76 wants to merge 1 commit into SvenWoltmann:main from Pankraz76:pmd

Pankraz76 commented Jan 2, 2025

Description

This Pull Request removes the explicit PMD ruleset configuration (pmd-ruleset.xml) and opts to rely on the default PMD configurations.

Key Reasons:

Simplification: By following the principle of "Convention Over Configuration," we reduce the need for custom rule management.
Best Practices: Default PMD settings align with commonly accepted standards and practices out-of-the-box.
Maintenance: Eliminates the overhead of maintaining a separate pmd-ruleset.xml file.

Changes Made

Deleted pmd-ruleset.xml.
Updated pom.xml to remove references to the deleted ruleset.

Impacts

Custom rules defined in pmd-ruleset.xml will no longer apply.
Builds and analyses will follow PMD's default behavior, ensuring alignment with community standards.

How to Test

Run mvn pmd:check to verify builds and PMD analysis with default settings.
Ensure the output aligns with expected code quality and rule coverage.


          use-COC-over-explicit-pmd-ruling

fa56328

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet