Skip to content

Commit

Permalink
Merge pull request #77 from pinax/django-20-compatibility
Browse files Browse the repository at this point in the history
Revise testing config
  • Loading branch information
grahamu authored Nov 1, 2017
2 parents f80f2b6 + 57ce39f commit 8f17f97
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 7 deletions.
14 changes: 8 additions & 6 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,12 @@ common: &common
- run:
name: upload coverage report
command: |
if [[ "$TOXENV" != checkqa ]]; then
PATH=$HOME/.local/bin:$PATH
pip install --user codecov
~/.local/bin/codecov --required --flags $CIRCLE_JOB
fi
if [[ "$UPLOAD_COVERAGE" != 0 ]]; then
PATH=$HOME/.local/bin:$PATH
pip install --user codecov
coverage xml
~/.local/bin/codecov --required -X search gcov pycov -f coverage.xml --flags $CIRCLE_JOB
fi
- save_cache:
paths:
- .tox
Expand All @@ -36,7 +37,8 @@ jobs:
docker:
- image: circleci/python:3.6.1
environment:
TOXENV=checkqa
- TOXENV=checkqa
- UPLOAD_COVERAGE=0
py27dj18:
<<: *common
docker:
Expand Down
3 changes: 2 additions & 1 deletion tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ inline-quotes = double
[isort]
multi_line_output=3
known_django=django
known_third_party=
known_third_party=pinax
sections=FUTURE,STDLIB,DJANGO,THIRDPARTY,FIRSTPARTY,LOCALFOLDER
skip_glob=*/pinax/notifications/migrations/*

Expand Down Expand Up @@ -41,6 +41,7 @@ deps =
dj110: Django>=1.10,<1.11
dj111: Django>=1.11,<1.12
dj20: Django<2.1
master: https://github.com/django/django/tarball/master

usedevelop = True
commands =
Expand Down

0 comments on commit 8f17f97

Please sign in to comment.