Skip to content

[blocked upstream] ci: use coveralls for all jobs (fixed for Coverage.py 5) #88

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
blueyed wants to merge 14 commits into
base: master
Choose a base branch
from
Open

[blocked upstream] ci: use coveralls for all jobs (fixed for Coverage.py 5) #88

Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
f0a9b27
ci: use coveralls for all jobs (fixed for Coverage.py 5)
blueyed Jan 2, 2020
628fd6d
ci: CircleCI: use .venv
blueyed Jan 2, 2020
8ada6ba
ci: CircleCI: upgrade pip
blueyed Jan 2, 2020
379ce8d
tox: --notest first, -vv
blueyed Jan 2, 2020
4484f46
upgrade virtualenv
blueyed Jan 2, 2020
33fe09a
back to 'pip install --user tox'
blueyed Jan 2, 2020
5a26f4d
fix comment
blueyed Jan 2, 2020
24c0eed
https://coveralls.io/webhook without repo_token
blueyed Jan 2, 2020
547825e
coveralls: notify via curl
blueyed Jan 2, 2020
75c2aa2
fixup! coveralls: notify via curl
blueyed Jan 2, 2020
49b3b2b
fixup! fixup! coveralls: notify via curl
blueyed Jan 2, 2020
bccf6fc
Revert "fixup! fixup! coveralls: notify via curl"
blueyed Jan 2, 2020
1e4a547
fixup! fixup! coveralls: notify via curl
blueyed Jan 2, 2020
871888b
fix notify
blueyed Jan 2, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 25 additions & 13 deletions .circleci/config.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@ common: &common
steps:
- checkout
- run: pip install --user tox
- run: ~/.local/bin/tox --version
- run: PYTEST_ADDOPTS=-vv ~/.local/bin/tox
- run: ~/.local/bin/tox -vv --notest
- run: PYTEST_ADDOPTS=-vv ~/.local/bin/tox -vv
- run:
name: upload coverage results for non-checkqa builds
command: |
Expand All @@ -25,14 +25,10 @@ common: &common

bash <(curl -S -L --connect-timeout 5 --retry 6 -s https://codecov.io/bash) -Z -X fix -f coverage.xml -F "${CIRCLE_JOB//-/,}"

if [[ "$CIRCLE_JOB" != py37-coveragepy5 ]]; then
# coveralls-python does not work with Coverage.py 5's data yet.
# (https://github.com/coveralls-clients/coveralls-python/issues/203)
pip install coveralls
COVERALLS_PARALLEL=true coveralls
fi
pip install coveralls
COVERALLS_PARALLEL=true coveralls

# Coveralls and Codacy do not support merged reports.
# Codacy does not support merged reports.
if [[ "$CIRCLE_JOB" == py36 ]]; then
pip install codacy-coverage
python-codacy-coverage --verbose --report coverage.xml
Expand Down Expand Up @@ -95,6 +91,16 @@ jobs:
environment:
TOXENV=checkqa

notify:
<<: *common
docker:
- image: circleci/python:3.8
steps:
- run:
name: Report finished build to coveralls
command: |
curl -k https://coveralls.io/webhook?repo_token=$COVERALLS_REPO_TOKEN -d "payload[build_num]=$CIRCLE_BUILD_NUM&payload[status]=done"

workflows:
version: 2
test:
Expand All @@ -108,7 +114,13 @@ workflows:
- py34
- py27
- checkqa

notify:
webhooks:
- url: https://coveralls.io/webhook?repo_token=$COVERALLS_REPO_TOKEN
- notify:
requires:
- py38
- py37-coveragepy5
- py37
- py37-click6
- py36
- py35
- py34
- py27