Skip to content

Commit b8ebca1

Browse files
committed
ci: remove Windows from CI matrices (bash tool not supported on Windows)
1 parent be966dd commit b8ebca1

1 file changed

Lines changed: 7 additions & 45 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 7 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ jobs:
1616
strategy:
1717
fail-fast: false
1818
matrix:
19-
os: [ubuntu-latest, macos-latest, windows-latest]
19+
os: [ubuntu-latest, macos-latest]
2020
steps:
2121
- uses: actions/checkout@v7
2222

@@ -41,7 +41,7 @@ jobs:
4141
strategy:
4242
fail-fast: false
4343
matrix:
44-
os: [ubuntu-latest, macos-latest, windows-latest]
44+
os: [ubuntu-latest, macos-latest]
4545
steps:
4646
- uses: actions/checkout@v7
4747

@@ -65,7 +65,7 @@ jobs:
6565
strategy:
6666
fail-fast: false
6767
matrix:
68-
os: [ubuntu-latest, macos-latest, windows-latest]
68+
os: [ubuntu-latest, macos-latest]
6969
python-version: ["3.11", "3.12", "3.13"]
7070
steps:
7171
- uses: actions/checkout@v7
@@ -108,7 +108,7 @@ jobs:
108108
strategy:
109109
fail-fast: false
110110
matrix:
111-
os: [ubuntu-latest, macos-latest, windows-latest]
111+
os: [ubuntu-latest, macos-latest]
112112
steps:
113113
- uses: actions/checkout@v7
114114

@@ -121,17 +121,14 @@ jobs:
121121
python -m pip install --upgrade pip build
122122
python -m build
123123
124-
# POSIX jobs: /tmp venv + sh smoke test.
125-
- name: Install the built wheel into a clean venv (POSIX)
126-
if: runner.os != 'Windows'
124+
- name: Install the built wheel into a clean venv
127125
run: |
128126
python -m venv /tmp/wheeltest
129127
# Install ONLY from the built wheel (no source tree / editable install)
130128
/tmp/wheeltest/bin/pip install --upgrade pip
131129
/tmp/wheeltest/bin/pip install dist/*.whl
132130
133-
- name: CLI smoke test (POSIX)
134-
if: runner.os != 'Windows'
131+
- name: CLI smoke test
135132
# Run from a neutral directory: if we ran inside the checkout, the
136133
# source `python_agent_harness/` tree would shadow the installed wheel
137134
# on sys.path and we'd be testing the source, not the built artifact.
@@ -158,40 +155,6 @@ jobs:
158155
print("bundled prompt OK:", agent)
159156
PY
160157
161-
# Windows jobs: %RUNNER_TEMP% venv + PowerShell smoke test.
162-
- name: Install the built wheel into a clean venv (Windows)
163-
if: runner.os == 'Windows'
164-
shell: pwsh
165-
run: |
166-
python -m venv "$env:RUNNER_TEMP\wheeltest"
167-
& "$env:RUNNER_TEMP\wheeltest\Scripts\pip.exe" install --upgrade pip
168-
& "$env:RUNNER_TEMP\wheeltest\Scripts\pip.exe" install (Get-Item dist\*.whl).FullName
169-
170-
- name: CLI smoke test (Windows)
171-
if: runner.os == 'Windows'
172-
working-directory: ${{ runner.temp }}
173-
shell: pwsh
174-
run: |
175-
$ErrorActionPreference = 'Stop'
176-
$Venv = "$env:RUNNER_TEMP\wheeltest"
177-
# Console script is installed and importable
178-
& "$Venv\Scripts\python-agent-harness.exe" --help
179-
# Config subcommand runs without any API key / network
180-
& "$Venv\Scripts\python-agent-harness.exe" config
181-
# Module entry point works too
182-
& "$Venv\Scripts\python.exe" -m python_agent_harness --help
183-
# Import resolves to the INSTALLED wheel + bundled prompts shipped
184-
& "$Venv\Scripts\python.exe" -c @"
185-
import importlib.resources as r
186-
import python_agent_harness
187-
loc = python_agent_harness.__file__
188-
assert 'site-packages' in loc, f'imported source, not wheel: {loc}'
189-
agent = r.files('python_agent_harness').joinpath('prompts/agent.md')
190-
assert agent.is_file(), 'bundled prompt missing from wheel'
191-
print('wheel import OK:', loc)
192-
print('bundled prompt OK:', agent)
193-
"@
194-
195158
- name: Upload distributions
196159
uses: actions/upload-artifact@v7
197160
with:
@@ -204,7 +167,7 @@ jobs:
204167
strategy:
205168
fail-fast: false
206169
matrix:
207-
os: [ubuntu-latest, macos-latest, windows-latest]
170+
os: [ubuntu-latest, macos-latest]
208171
steps:
209172
- uses: actions/checkout@v7
210173

@@ -224,7 +187,6 @@ jobs:
224187
# POSIX-only shell: the ignore-list parsing is bash syntax, and the
225188
# audit result is OS-independent, so Windows skips this job.
226189
- name: pip-audit
227-
if: runner.os != 'Windows'
228190
shell: bash
229191
run: |
230192
IGNORE_VULNS=""

0 commit comments

Comments
 (0)