Skip to content

Commit ce6eeee

Browse files
committed
make coveralls report merge py2 and py3
new version uses different job ID, making the coveralls.io think that two runs were performed use common version across all versions to make sure merging them all works
1 parent c6a27b9 commit ce6eeee

File tree

2 files changed

+10
-8
lines changed

2 files changed

+10
-8
lines changed

.github/workflows/ci.yml

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -329,14 +329,16 @@ jobs:
329329
needs: test
330330
runs-on: ubuntu-latest
331331
steps:
332-
- name: Set up Python 3.8
333-
uses: actions/setup-python@v2
334-
with:
335-
python-version: 3.8
336332
- name: Finished
337333
env:
338334
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
339-
COVERALLS_SERVICE_NAME: git hub actions
335+
COVERALLS_REPO_TOKEN: ${{ secrets.COVERALLS_REPO_TOKEN }}
340336
run: |
341-
pip install --upgrade coveralls
342-
coveralls --service=github --finish
337+
if [[ "${{ github.event.pull_request }}" ]]; then
338+
PR_NUM=${GITHUB_REF#refs/pull/}
339+
PR_NUM=${PR_NUM%/merge}
340+
BUILD_NUM=${GITHUB_SHA}-PR-${PR_NUM}
341+
else
342+
BUILD_NUM=${GITHUB_SHA}
343+
fi
344+
curl -k https://coveralls.io/webhook?repo_token=$COVERALLS_REPO_TOKEN -d "payload[build_num]=${BUILD_NUM}&payload[status]=done"

build-requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
coverage
22
hypothesis
3-
coveralls
3+
coveralls==1.11.1
44
pylint
55
diff_cover
66
pytest>=4.6.5

0 commit comments

Comments
 (0)