[ 
https://issues.apache.org/jira/browse/NIFI-14859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18013784#comment-18013784
 ] 

Daniel Stieglitz edited comment on NIFI-14859 at 8/13/25 10:19 PM:
-------------------------------------------------------------------

[~larsfrancke] In Intellij I ran the
{code:java}
Missing '@Override' annotation Inspection{code}
and I see what you are talking about that there many more which are missing, 
although its mainly for unit and integration tests.


was (Author: JIRAUSER294662):
[~larsfrancke] In Intellij I ran the
{code:java}
Missing '@Override' annotation Inspection{code}
and I see what you are talking about that there many more which are missing 
although its mainly for unit and integration tests.

> contrib-check profile fails due to PMD errors
> ---------------------------------------------
>
>                 Key: NIFI-14859
>                 URL: https://issues.apache.org/jira/browse/NIFI-14859
>             Project: Apache NiFi
>          Issue Type: Bug
>    Affects Versions: 2.5.0
>            Reporter: Lars Francke
>            Assignee: Lars Francke
>            Priority: Minor
>
> The [pull request 
> template|https://github.com/apache/nifi/blob/main/.github/PULL_REQUEST_TEMPLATE.md]
>  suggests running {{{}mvn clean install -P contrib-check{}}}.
> (Aside: It may make sense to adjust the PR template to use the mvn wrapper 
> instead to get the intendend maven version)
> On current main this fails due to various PMD warnings.
> Excerpt:
> {code:java}
> [WARNING] PMD Failure: 
> org.apache.nifi.web.search.attributematchers.ConnectivityMatcherTest:38 
> Rule:MissingOverride Priority:3 The method 'setUp()' is missing an @Override 
> annotation..
> [WARNING] PMD Failure: 
> org.apache.nifi.web.search.attributematchers.ParameterMatcherTest:35 
> Rule:MissingOverride Priority:3 The method 'setUp()' is missing an @Override 
> annotation..
> [WARNING] PMD Failure: 
> org.apache.nifi.web.search.attributematchers.PrioritiesMatcherTest:40 
> Rule:MissingOverride Priority:3 The method 'setUp()' is missing an @Override 
> annotation..
> [WARNING] PMD Failure: 
> org.apache.nifi.web.search.attributematchers.ProcessGroupMatcherTest:33 
> Rule:MissingOverride Priority:3 The method 'setUp()' is missing an @Override 
> annotation.. {code}
>  I have not fixed all of them yet but there are more of these.
> This aborts the build.
> There are [CI 
> builds|https://github.com/apache/nifi/blob/52f060835dd36c0d0cd532637ce36f2fd0d76582/.github/workflows/code-compliance.yml#L55-L62]
>  using the contrib-check profile but they only run the [validate phase 
> |https://maven.apache.org/guides/introduction/introduction-to-the-lifecycle.html]
>  which does not trigger the PMD run.
> This has probably been in place since 
> https://issues.apache.org/jira/browse/NIFI-14821 and similar issues /cc 
> [~dstiegli1] 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to