Skip to content

Commit

Permalink
Merge pull request #458 from rollbar/remove-tox-from-ci
Browse files Browse the repository at this point in the history
Removed tox from CI and called unittest directly.
  • Loading branch information
danielmorell authored Sep 20, 2024
2 parents 4956f85 + ea2b619 commit b0d5a23
Showing 1 changed file with 4 additions and 9 deletions.
13 changes: 4 additions & 9 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,9 @@ jobs:
with:
python-version: ${{ matrix.python-version }}

- name: Install Python Test dependencies
run: pip install requests webob blinker httpx

- name: Install Python 3.6 dependencies
if: ${{ contains(matrix.python-version, '3.6') }}
# typing-extensions dropped support for Python 3.6 in version 4.2
Expand Down Expand Up @@ -132,13 +135,5 @@ jobs:
if: ${{ contains(matrix.framework, 'FASTAPI_VERSION') }}
run: pip install fastapi==$FASTAPI_VERSION

- name: Install Tox
run: pip install tox

- name: Run tests (skip on Python 3.6)
if: ${{ !contains(matrix.python-version, '3.6') }}
run: tox

- name: Run tests (Python 3.6)
if: ${{ contains(matrix.python-version, '3.6') }}
- name: Run tests
run: python -m unittest rollbar.test.discover

0 comments on commit b0d5a23

Please sign in to comment.