Skip to content

Commit 2e7e5b6

Browse files
authored
🧪 Drop venv when making coverage XML @ CI
This deletes the fallback that would use `--venv` when `--docker` is unavailable in `ansible-test`. But this option is present in all supported versions of Ansible as it was added in v2.12 which is EOL already. PR ansible#84819
1 parent 9821a4d commit 2e7e5b6

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

.azure-pipelines/scripts/report-coverage.sh

+3-3
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,6 @@ if ! ansible-test --help >/dev/null 2>&1; then
1212
pip install https://github.com/ansible/ansible/archive/devel.tar.gz --disable-pip-version-check
1313
fi
1414

15-
# Generate stubs using docker (if supported) otherwise fall back to using a virtual environment instead.
16-
# The use of docker is required when Powershell code is present, but Ansible 2.12 was the first version to support --docker with coverage.
17-
ansible-test coverage xml --group-by command --stub --docker --color -v || ansible-test coverage xml --group-by command --stub --venv --color -v
15+
# Generate stubs using docker.
16+
# The use of docker is mandatory when Powershell code is present.
17+
ansible-test coverage xml --group-by command --stub --docker --color -v

0 commit comments

Comments
 (0)