Skip to content

Commit a087250

Browse files
fix: add Windows CI matrix and install.ps1 syntax validation by reviewer-B
1 parent 45d082c commit a087250

2 files changed

Lines changed: 9 additions & 1 deletion

File tree

.github/workflows/ci.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,10 @@ jobs:
4848
echo "SKILL.md OK"
4949
5050
test:
51-
runs-on: ubuntu-latest
51+
strategy:
52+
matrix:
53+
os: [ubuntu-latest, windows-latest]
54+
runs-on: ${{ matrix.os }}
5255
steps:
5356
- uses: actions/checkout@v4
5457
- name: Run validation tests

tests/validate.sh

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,5 +32,10 @@ done
3232
# CHANGELOG.md version header
3333
grep -qE '^## \[[0-9]+\.[0-9]+\.[0-9]+\]' "$ROOT/CHANGELOG.md" || { echo "FAIL: CHANGELOG.md no version header"; fail=1; }
3434

35+
# Validate install.ps1 syntax (if pwsh available)
36+
if command -v pwsh &>/dev/null; then
37+
pwsh -NoProfile -Command "\$e=@(); [System.Management.Automation.PSParser]::Tokenize((Get-Content -Raw install.ps1),[ref]\$e)|Out-Null; if(\$e.Count){exit 1}" 2>/dev/null || { echo "FAIL: install.ps1 syntax error"; fail=1; }
38+
fi
39+
3540
echo "=== $([ "$fail" -eq 0 ] && echo 'ALL PASSED' || echo 'SOME FAILED') ==="
3641
exit $fail

0 commit comments

Comments
 (0)