You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@libcloud.apache.org by Kami <gi...@git.apache.org> on 2017/09/10 20:42:25 UTC

[GitHub] libcloud pull request #1109: [WIP] Replace coveralls with codecov.io for cod...

GitHub user Kami opened a pull request:

    https://github.com/apache/libcloud/pull/1109

    [WIP] Replace coveralls with codecov.io for code coverage reports

    This pull request proposes to replace coveralls coverage reports with codecov.io coverage reports.
    
    My main reasons for the proposed change of service are:
    
    * codecov.io has a much nicer and better UI.
    * Right now we have coverage comments enabled which means coveralls posts a comment on the pull request on every push. Imo, this adds a lot of noise, but very little value. It discourages people to push often (which is a good practice) and adds noise to the pull request which makes actual valuable review feedback harder to see / find.
    
    Codecov.io can be configured to add only a single comment which is then updated when code and coverage changes. Imo, that's much better than comment spam on every push.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/Kami/libcloud codecov_experiment

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/libcloud/pull/1109.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1109
    
----
commit 787a4f637ed10f619f89dfed58aeaa488fbc065f
Author: Tomaz Muraus <to...@tomaz.me>
Date:   2017-09-10T20:36:05Z

    Enable coverage reports on codecov.

----


---

[GitHub] libcloud pull request #1109: [WIP] Replace coveralls with codecov.io for cod...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/libcloud/pull/1109


---