Skip to content

Commit

Permalink
Remove Coveralls
Browse files Browse the repository at this point in the history
This pull request removes the coveralls service integration since it
causes problems with running tests on pull requests. Instead, we just
use `coverage` with a limit of 91% coverage. That should do the work
and is much simpler.
  • Loading branch information
lkiesow committed Jan 3, 2024
1 parent 03b5e90 commit ac8641f
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 8 deletions.
1 change: 0 additions & 1 deletion .github/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
bandit
coverage
coveralls
flake8
mock
selenium>=4.13.0
6 changes: 2 additions & 4 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,5 @@ jobs:
- name: run user interface integration tests
run: ./.github/selenium-tests

- name: upload test coverage to coveralls
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
run: coveralls --service=github
- name: ensure test coverage doesn't drop
run: coverage report --fail-under=91
3 changes: 0 additions & 3 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,6 @@ PyCA – Opencast Capture Agent
.. image:: https://github.com/opencast/pyCA/workflows/Test%20pyCA/badge.svg?branch=master
:target: https://github.com/opencast/pyCA/actions?query=workflow%3A%22Test+pyCA%22+branch%3Amaster
:alt: Test pyCA GitHub Workflow Status
.. image:: https://coveralls.io/repos/github/opencast/pyCA/badge.svg?branch=master
:target: https://coveralls.io/github/opencast/pyCA?branch=master
:alt: Test Coverage
.. image:: https://img.shields.io/github/license/opencast/pyCA
:target: https://github.com/opencast/pyCA/blob/master/license.lgpl
:alt: LGPL-3 license
Expand Down

0 comments on commit ac8641f

Please sign in to comment.