Skip to content

Commit 9ddb343

Browse files
HanSur94claude
andcommitted
fix(ci): use pip install -e on Windows instead of install.bat
install.bat creates a venv with potentially stale packages. Direct pip install -e ensures the current source is used. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 9d59690 commit 9ddb343

1 file changed

Lines changed: 3 additions & 18 deletions

File tree

.github/workflows/ci.yml

Lines changed: 3 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -78,26 +78,11 @@ jobs:
7878
- uses: actions/setup-python@v5
7979
with:
8080
python-version: ${{ matrix.python-version }}
81-
- name: Run install.bat
82-
shell: cmd
83-
run: |
84-
call install.bat < nul
81+
- run: pip install -e ".[dev,monitoring]"
8582
- name: Verify MCP server module loads
86-
shell: cmd
87-
run: |
88-
call .venv\Scripts\activate.bat
89-
python -c "from matlab_mcp.server import main; print('OK: server module loads')"
90-
- name: Verify monitoring subpackage
91-
shell: cmd
92-
run: |
93-
call .venv\Scripts\activate.bat
94-
python -c "from matlab_mcp.monitoring.collector import MetricsCollector; print('OK: monitoring loads')"
83+
run: python -c "from matlab_mcp.server import main; print('OK')"
9584
- name: Run tests (no MATLAB engine)
96-
shell: cmd
97-
run: |
98-
call .venv\Scripts\activate.bat
99-
pip install pytest pytest-asyncio pytest-cov --quiet
100-
pytest tests/ -v -k "not matlab" --ignore=tests/test_integration.py --ignore=tests/test_mcp_integration.py -W ignore::pytest.PytestUnraisableExceptionWarning
85+
run: pytest tests/ -v -k "not matlab" --ignore=tests/test_integration.py --ignore=tests/test_mcp_integration.py -W ignore::pytest.PytestUnraisableExceptionWarning
10186

10287
test-macos:
10388
needs: lint

0 commit comments

Comments
 (0)