Skip to content

Commit fa76a5c

Browse files
committed
fix: update CI to use python -m for all commands
- Changed pytest, ruff, and mypy to use 'python -m' prefix - This ensures commands are found in the Python environment - Removed duplicate coverage options from pyproject.toml that conflicted with CI - Exit code 2 was likely due to pytest command not being in PATH
1 parent ce3746d commit fa76a5c

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

.github/workflows/ci.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -36,14 +36,14 @@ jobs:
3636
3737
- name: Run linting with ruff
3838
run: |
39-
ruff check .
40-
ruff format --check .
39+
python -m ruff check .
40+
python -m ruff format --check .
4141
4242
- name: Run type checking with mypy
43-
run: mypy src tests
43+
run: python -m mypy src tests
4444

4545
- name: Run tests with pytest
46-
run: pytest -v --cov=nutrient_dws --cov-report=xml --cov-report=term
46+
run: python -m pytest -v --cov=nutrient_dws --cov-report=xml --cov-report=term
4747

4848
- name: Upload coverage to Codecov
4949
uses: codecov/codecov-action@v4

pyproject.toml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -112,10 +112,6 @@ testpaths = ["tests"]
112112
addopts = [
113113
"-ra",
114114
"--strict-markers",
115-
"--cov=nutrient_dws",
116-
"--cov-report=term-missing",
117-
"--cov-report=html",
118-
"--cov-report=xml",
119115
]
120116
markers = [
121117
"integration: marks tests as integration tests (deselect with '-m \"not integration\"')",

0 commit comments

Comments
 (0)