Skip to content

Commit

Permalink
Temporary disable xdist on GHA
Browse files Browse the repository at this point in the history
Related: #3568
  • Loading branch information
ssbarnea committed Jun 15, 2023
1 parent 18614d5 commit ab9a77c
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 8 deletions.
1 change: 1 addition & 0 deletions .config/dictionary.txt
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ EPIPE # linux
Fimport
Jython
MYTAG
PYTEST_ADDOPS
PYTHONBREAKPOINT
PYTHONIOENCODING
PYTHONPYCACHEPREFIX
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/tox.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,9 @@ concurrency:
env:
FORCE_COLOR: 1 # tox, pytest, ansible-lint
PY_COLORS: 1
# Temporary disable parallelism under GHA until we sort concurrency issues
# related to collection installations.
PYTEST_ADDOPS: "-n1"

jobs:
pre:
Expand Down
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,7 @@ include = ["src"]
[tool.pytest.ini_options]
# do not add options here as this will likely break either console runs or IDE
# integration like vscode or pycharm
addopts = "-p no:pytest_cov"
addopts = "-p no:pytest_cov -n auto -ra --showlocals --doctest-modules --durations=10"
# https://code.visualstudio.com/docs/python/testing
# coverage is re-enabled in `tox.ini`. That approach is safer than
# `--no-cov` which prevents activation from tox.ini and which also fails
Expand Down
8 changes: 1 addition & 7 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -29,13 +29,7 @@ commands_pre =
commands =
# safety measure to assure we do not accidentally run tests with broken dependencies
{envpython} -m pip check
coverage run -m pytest {posargs:\
-n auto \
-ra \
--showlocals \
--doctest-modules \
--durations=10 \
}
coverage run -m pytest {posargs:}
sh -c "coverage combine -a -q --data-file=.coverage .tox/.coverage.*"

passenv =
Expand Down

0 comments on commit ab9a77c

Please sign in to comment.