Skip to content

Commit

Permalink
Add Coveralls parallel build coverage
Browse files Browse the repository at this point in the history
Resolves #180

**Why this change was necessary**
We want to get an aggregate coverage report for each matrix build
option.

**What this change does**
 - Names coverage reports to indicate OS and Python version.
 - Runs a Coveralls finishing step to let Coveralls know the parallel
 build is finished.

**Any side-effects?**
None

**Additional context/notes/links**
https://coveralls-python.readthedocs.io/en/latest/usage/configuration.html#github-actions-gotcha
  • Loading branch information
jidicula committed Nov 29, 2020
1 parent 8692c85 commit e5b5cbe
Showing 1 changed file with 21 additions and 5 deletions.
26 changes: 21 additions & 5 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,25 @@ jobs:
- name: Run CI script
run: ./ci.sh

- name: Coveralls
if: contains(matrix.os, 'ubuntu-18.04')
- name: Upload Coverage
run: coveralls
env:
GITHUB_TOKEN: ${{ secrets.github_token }}
run: |
coveralls
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
COVERALLS_FLAG_NAME: ${{ matrix.os }}-Python-${{ matrix.python-version }}
COVERALLS_PARALLEL: true
coveralls:
name: Finish Coveralls
needs: test
runs-on: 'ubuntu-18.04'
steps:

- name: Set up Python
uses: actions/setup-python@v2
with:
python-version: '3.7'
- name: Finished
run: |
pip3 install --upgrade coveralls
coveralls --finish
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

0 comments on commit e5b5cbe

Please sign in to comment.