Skip to content

Commit

Permalink
ci: update the TIOBE reporting for the changes in coverage calculation (
Browse files Browse the repository at this point in the history
#1367)

We recently changed `tox -e unit` to not produce a coverage report. This
PR adjust the TIOBE workflow to account for that - running `tox -e
coverage` instead.

Since this will now be one of the main uses for that command, I've also
moved the `coverage xml` command into the tox steps rather than being in
the workflow, and adjusted the `.gitignore` accordingly.
  • Loading branch information
tonyandrewmeyer authored Sep 9, 2024
1 parent 7c51afa commit 107fdd7
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/tiobe.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,7 @@ jobs:
run: pip install tox~=4.2 coverage[toml] flake8 pylint websocket-client==1.* pyyaml==6.* pytest~=7.2 pytest-operator~=0.23
- name: Generate coverage report
run: |
tox -e unit
coverage xml
tox -e coverage
# Annoyingly, the coverage.xml file needs to be in a .coverage folder.
rm .coverage
mkdir .coverage
Expand Down
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ venv
/docsenv
.vscode
.coverage
coverage.xml
/.tox
.*.swp

Expand Down
1 change: 1 addition & 0 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -113,6 +113,7 @@ deps =
commands =
coverage run --source={[vars]src_path} \
-m pytest --ignore={[vars]tst_path}smoke -v --tb native {posargs}
coverage xml
coverage report

[testenv:pebble]
Expand Down

0 comments on commit 107fdd7

Please # to comment.