[ https://issues.apache.org/jira/browse/MPMD-383?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17906431#comment-17906431 ]
Andreas Dangel commented on MPMD-383: ------------------------------------- Agreed. In practice, PMD would be turned off if it fails the build. Thanks for the reality check. Where would this leave us in terms of conclusion for this issue? The desired behavior ("PMD should fail when there is a processing error") can already been switched on via property {{skipPmdError}}. Changing the default behavior to fail on processing errors would only make sense in an ideal world (where build errors are fixed in the tools instead of switching off tools). Does that mean, we can close this issue as "won't fix"? > PMD should fail when there is a processing error > ------------------------------------------------ > > Key: MPMD-383 > URL: https://issues.apache.org/jira/browse/MPMD-383 > Project: Maven PMD Plugin > Issue Type: Wish > Components: PMD > Affects Versions: 3.21.0 > Reporter: Thomas Nikolay > Priority: Major > Attachments: image-2023-07-28-13-34-28-717.png > > > We had the case that PMD can't process some processing error but the build > does not fail. > > {noformat} > [2023-07-28T10:59:44.102Z] [INFO] --- maven-pmd-plugin:3.20.0:pmd (pmd) @ > mod-config-client-cst --- > [2023-07-28T10:59:44.692Z] [INFO] PMD version: 6.53.0 > [2023-07-28T10:59:45.672Z] [WARNING] Removed misconfigured rule: > GK-SAP-ApacheLoggerDeniedRule cause: missing value > [2023-07-28T10:59:45.672Z] [WARNING] There are 3 PMD processing errors: > [2023-07-28T10:59:45.672Z] [WARNING] > /var/lib/hudson-slave/workspace/hmr.MAIN.00-FEATURE-BRANCHES/pos/config/mod-config-client-cst/src/test/java/com/gk_software/cst/client/AppInfoProviderImpl_CST_Test.java: > PMDException: Error while processing > /var/lib/hudson-slave/workspace/hmr.MAIN.00-FEATURE-BRANCHES/pos/config/mod-config-client-cst/src/test/java/com/gk_software/cst/client/AppInfoProviderImpl_CST_Test.java > [2023-07-28T10:59:45.948Z] > /var/lib/hudson-slave/workspace/hmr.MAIN.00-FEATURE-BRANCHES/pos/config/mod-config-client-cst/src/test/java/com/gk_software/pos/api/model/config/component/client/print/additional_printout/Trigger_CFH_CST_Test.java: > PMDException: Error while processing > /var/lib/hudson-slave/workspace/hmr.MAIN.00-FEATURE-BRANCHES/pos/config/mod-config-client-cst/src/test/java/com/gk_software/pos/api/model/config/component/client/print/additional_printout/Trigger_CFH_CST_Test.java > [2023-07-28T10:59:45.948Z] > /var/lib/hudson-slave/workspace/hmr.MAIN.00-FEATURE-BRANCHES/pos/config/mod-config-client-cst/src/main/java/com/gk_software/pos/api/model/config/component/client/print/additional_printout/Trigger_CFH_CST.java: > PMDException: Error while processing > /var/lib/hudson-slave/workspace/hmr.MAIN.00-FEATURE-BRANCHES/pos/config/mod-config-client-cst/src/main/java/com/gk_software/pos/api/model/config/component/client/print/additional_printout/Trigger_CFH_CST.java > [2023-07-28T10:59:45.948Z] [INFO] Rendering content with > org.apache.maven.skins:maven-default-skin:jar:1.3 skin. > [2023-07-28T10:59:46.210Z] [INFO] > [2023-07-28T10:59:46.210Z] [INFO] <<< maven-pmd-plugin:3.20.0:check > (pmd-check) < :pmd @ mod-config-client-cst <<< > {noformat} > > My expectation would be that the build failed when there a processing errors. > -- This message was sent by Atlassian Jira (v8.20.10#820010)