Skip to content

Commit

Permalink
Avoid duplicate test runs in CI (wpilibsuite#295)
Browse files Browse the repository at this point in the history
Tox is running the tests for every Python instance listed in tox.ini
when CI should just be calling pytest.
  • Loading branch information
calcmogul authored Nov 14, 2024
1 parent 8458230 commit 7c04d3b
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 17 deletions.
8 changes: 3 additions & 5 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,7 @@ jobs:
with:
python-version: ${{ matrix.python-version }}

- name: Install Python dependencies
run: pip install build
- run: pip install build

- name: Install wpiformat
run: |
Expand All @@ -39,15 +38,14 @@ jobs:
pip install dist/*.whl
shell: bash

- name: Install test dependencies
run: pip install tox
- run: pip install pytest

- name: Run unit tests
run: |
git config --global user.email "[email protected]"
git config --global user.name "Your Name"
cd wpiformat
tox
pytest
- name: wpiformat - whole repo
run: |
Expand Down
12 changes: 0 additions & 12 deletions wpiformat/tox.ini

This file was deleted.

0 comments on commit 7c04d3b

Please sign in to comment.