fix: Correct order of Codecov reporting in CI (#711)

* Report coverage of src with Codecov before the coverage.xml is overwritten by testing pyhf.contrib
   - Amends PR #694
* Update Codecov GitHub Actions app to v1.0.5

Author: matthewfeickert

.github/workflows/ci.yml

@@ -42,17 +42,17 @@ jobs:
     - name: Test with pytest
       run: |
         python -m pytest -r sx --ignore tests/benchmarks/ --ignore tests/contrib --ignore tests/test_notebooks.py
-    - name: Test Contrib
-      if: matrix.python-version != 2.7
-      run: |
-        python -m pytest -r sx tests/contrib --mpl --mpl-baseline-path tests/contrib/baseline
     - name: Report coverage with Codecov
       if: github.event_name == 'push' && matrix.python-version == 3.7 && matrix.os == 'ubuntu-latest'
-      uses: codecov/codecov-action@v1.0.4
+      uses: codecov/codecov-action@v1.0.5
       with:
         token: ${{ secrets.CODECOV_TOKEN }}
         file: ./coverage.xml
         flags: unittests
+    - name: Test Contrib module with pytest
+      if: matrix.python-version != 2.7
+      run: |
+        python -m pytest -r sx tests/contrib --mpl --mpl-baseline-path tests/contrib/baseline
     - name: Run benchmarks
       if: github.event_name == 'schedule' && matrix.python-version == 3.7
       run: |