Repository: joblib/joblib
Pull request #492 (2017-02-06 09:25:28)

[MRG] Setup integration with codecov (and remove coveralls)

ogrisel: This seems to work as expected. Maybe we can remove coveralls.
aabadie: > Maybe we can remove coveralls.

+1 for removing coveralls
ogrisel: Feel free to do that in this PR then.
aabadie: All green. If everybody agrees, this can be squashed and rebased
lesteve: There is still a coveralls mention somewhere:
```
❯ git grep -i coverall
continuous_integration/test_script.sh:        # has to generate report for coveralls ...
```
aabadie: > There is still a coveralls mention somewhere

Thanks ! replaced.

Patch:
diff --git a/.travis.yml b/.travis.yml
index b6524a3..4c6c1cb 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -26,9 +26,7 @@ script:
     - source continuous_integration/test_script.sh
 
 after_success:
-    # Ignore coveralls failures as the coveralls server is not very reliable
-    # but we don't want travis to report a failure in the github UI just
-    # because the coverage report failed to be published.
-    # coveralls need to be run from the git checkout
-    # so we need to copy the coverage results from TEST_RUN_FOLDER
-    - if [[ "$COVERAGE" == "true" ]]; then coveralls || echo "failed"; fi
+    # Ignore codecov failures because we don't want travis to report a
+    # failure in the github UI just because the coverage report failed
+    # to be published.
+    - if [[ "$COVERAGE" == "true" ]]; then codecov || echo "failed"; fi
diff --git a/README.rst b/README.rst
index 0f729f2..814fc44 100644
--- a/README.rst
+++ b/README.rst
@@ -50,7 +50,7 @@ Run the test suite using::
 
 from the root of the project.
 
-|Travis| |AppVeyor| |Coveralls|
+|Travis| |AppVeyor| |Codecov|
 
 .. |Travis| image:: https://travis-ci.org/joblib/joblib.svg?branch=master
    :target: https://travis-ci.org/joblib/joblib
@@ -60,9 +60,10 @@ from the root of the project.
    :target: https://ci.appveyor.com/project/joblib-ci/joblib/history
    :alt: AppVeyor build status
 
-.. |Coveralls| image:: https://coveralls.io/repos/joblib/joblib/badge.svg?branch=master&service=github
-   :target: https://coveralls.io/github/joblib/joblib?branch=master
-   :alt: Coveralls coverage
+.. |Codecov| image:: https://codecov.io/gh/joblib/joblib/branch/master/graph/badge.svg
+   :target: https://codecov.io/gh/joblib/joblib
+   :alt: Codecov coverage
+
 
 Building the docs
 =========================
diff --git a/continuous_integration/install.sh b/continuous_integration/install.sh
index 6de44a2..b8811e6 100755
--- a/continuous_integration/install.sh
+++ b/continuous_integration/install.sh
@@ -81,7 +81,7 @@ if [ -n "$NUMPY_VERSION" ]; then
 fi
 
 if [[ "$COVERAGE" == "true" ]]; then
-    pip install pytest-cov coverage coveralls
+    pip install pytest-cov codecov
 fi
 
 python setup.py install


Test patch:
diff --git a/continuous_integration/test_script.sh b/continuous_integration/test_script.sh
index 2eba59c..e7b0816 100755
--- a/continuous_integration/test_script.sh
+++ b/continuous_integration/test_script.sh
@@ -17,7 +17,7 @@ if [[ "$SKIP_TESTS" != "true" ]]; then
 
     if [ "$COVERAGE" == "true" ]; then
         # Add coverage option to setup.cfg file if current test run
-        # has to generate report for coveralls ...
+        # has to generate report for codecov ...
         export PYTEST_ADDOPTS="--cov=joblib"
     fi
     make