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

Zili Chen commented on CURATOR-660:
-----------------------------------

[~tamaas] [~andor] [~dora.horvath] Many projects have such scripts or 
integrations, but few contributors/reviewers really take it into account when 
deciding whether or not to merge a patch.

See https://github.com/apache/zookeeper/pull/1905 sonatype-lift complains a lot 
but we don't care about.
See more on https://github.com/apache/pulsar/pulls codecov comments on every 
PR, but no contributor or reviewer care about the report.

However, adding such scripts do no harm, so I can merge the patch and see if 
the situation is different to Curator. Also, the origin patch isn't integrated 
into an automation workflow so I doubt more about its benefit.

> Enable code coverage reporting to SonarQube
> -------------------------------------------
>
>                 Key: CURATOR-660
>                 URL: https://issues.apache.org/jira/browse/CURATOR-660
>             Project: Apache Curator
>          Issue Type: Task
>            Reporter: Dóra Horváth
>            Assignee: Zili Chen
>            Priority: Minor
>              Labels: pull-request-available
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Enable coverage report generation and publishing to SonarQube.



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

Reply via email to