Skip to content

Commit

Permalink
Remove codecoverage analysis
Browse files Browse the repository at this point in the history
There is very little value for these reports in a project like robottelo
They detract from the overall effect of a PR's other checks
With that in mind, I'm removing code coverage checks with this PR until
further notice.
  • Loading branch information
JacobCallahan authored and pondrejk committed Jan 3, 2022
1 parent 6772c9c commit 22ab55c
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 18 deletions.
9 changes: 0 additions & 9 deletions .github/workflows/merge_to_master.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,9 +49,6 @@ jobs:
pytest --collect-only --disable-pytest-warnings -m pre_upgrade tests/upgrades/
pytest --collect-only --disable-pytest-warnings -m post_upgrade tests/upgrades/
- name: Test Robottelo Coverage
run: pytest --cov --cov-config=.coveragerc --cov-report=xml tests/robottelo

- name: Make Docs
run: |
make test-docstrings
Expand All @@ -61,12 +58,6 @@ jobs:
if: failure()
run: git diff

- name: Upload Codecov Coverage
uses: codecov/[email protected]
with:
file: coverage.xml
name: ${{ github.run_id }}-py-${{ matrix.python-version }}


robottelo_container:
needs: codechecks
Expand Down
9 changes: 0 additions & 9 deletions .github/workflows/pull_request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -54,9 +54,6 @@ jobs:
pytest --collect-only --disable-pytest-warnings -n 2 -m pre_upgrade tests/upgrades/
pytest --collect-only --disable-pytest-warnings -n 2 -m post_upgrade tests/upgrades/
- name: Test Robottelo Coverage
run: pytest --cov --cov-config=.coveragerc --cov-report=xml tests/robottelo

- name: Make Docs
run: |
make test-docstrings
Expand All @@ -65,9 +62,3 @@ jobs:
- name: Analysis (git diff)
if: failure()
run: git diff

- name: Upload Codecov Coverage
uses: codecov/[email protected]
with:
file: coverage.xml
name: ${{ github.run_id }}-py-${{ matrix.python-version }}

0 comments on commit 22ab55c

Please sign in to comment.