Skip to content

Commit

Permalink
fix: tox py3.7 & py3.8 failing tests in CI (#133)
Browse files Browse the repository at this point in the history
* py37 & py38 check fix

* review changes
  • Loading branch information
Ruchip16 committed Jun 21, 2023
1 parent 150dfe7 commit 8065723
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 10 deletions.
14 changes: 10 additions & 4 deletions .config/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,17 +1,23 @@
#
# This file is autogenerated by pip-compile with Python 3.7
# This file is autogenerated by pip-compile with Python 3.9
# by the following command:
#
# pip-compile --extra=docs --extra=test --no-annotate --output-file=.config/requirements.txt --resolver=backtracking --strip-extras --unsafe-package=ruamel-yaml-clib pyproject.toml
#
ansible-core==2.15.1
attrs==22.2.0
cffi==1.15.1
coverage==7.2.2
cryptography==41.0.1
exceptiongroup==1.1.1
importlib-metadata==6.6.0
importlib-resources==5.0.7
iniconfig==2.0.0
jinja2==3.1.2
markupsafe==2.1.3
packaging==23.0
pluggy==1.0.0
pycparser==2.21
pytest==7.2.2
pyyaml==6.0
resolvelib==1.0.1
tomli==2.0.1
typing-extensions==4.5.0
zipp==3.15.0
5 changes: 0 additions & 5 deletions .github/workflows/tox.yml
Original file line number Diff line number Diff line change
Expand Up @@ -100,11 +100,6 @@ jobs:
- name: tox -e ${{ matrix.passed_name }}
run: python3 -m tox -e ${{ matrix.passed_name }}

- name: Combine coverage data
if: ${{ startsWith(matrix.passed_name, 'py') }}
# produce a single .coverage file at repo root
run: tox -e coverage

- name: Upload coverage data
if: ${{ startsWith(matrix.passed_name, 'py') }}
uses: codecov/codecov-action@v3
Expand Down
6 changes: 5 additions & 1 deletion tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,9 @@ envlist =
lint,
pkg,
py,
devel
devel,
py37,
py38
skip_missing_interpreters = true

[testenv]
Expand All @@ -21,6 +23,8 @@ setenv =
ANSIBLE_LOCAL_TEMP = {envdir}/.ansible-local
PIP_CONSTRAINT = {toxinidir}/.config/requirements.txt
devel: PIP_CONSTRAINT = /dev/null
py37: PIP_CONSTRAINT = /dev/null
py38: PIP_CONSTRAINT = /dev/null
extras =
test
allowlist_externals =
Expand Down

0 comments on commit 8065723

Please sign in to comment.