Github user MikeThomsen commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/2615#discussion_r192169823
  
    --- Diff: 
nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-client-service/pom.xml
 ---
    @@ -212,6 +212,31 @@
                         </execution>
                     </executions>
                 </plugin>
    +
    +            <plugin>
    +                <groupId>org.jacoco</groupId>
    --- End diff --
    
    I'm not sure how well that would work at root level because there are 
plenty of integration tests that have to be run to get a full sense of code 
coverage. So maybe I should back this out or one of you can drop it when 
rebasing for a merge if you think it makes more sense to add a root level 
profile for code coverage.


---

Reply via email to