Github user phunt commented on the issue:

    https://github.com/apache/zookeeper/pull/467
  
    +1 lgtm. I was able to run the coverage successfully and view the report. 
    
    possible followups:
    
    1) The report/results doesn't look great - only 50% coverage.
    
    2) I had to install lcov in order to run this. It would be a good idea to 
update the docs - probably the c README - with details on how to run coverage 
and what the requirements are.
    
    3) What about the cmake build? Do we need to do anything there? (if not 
perhaps some docs?)


---

Reply via email to