Skip to content

ci: run the CI on Windows beside Linux - #136

Merged
Upd4ting merged 2 commits into
mainfrom
ci/windows
Nov 13, 2026
Merged

Upd4ting merged 2 commits into
mainfrom
ci/windows

Conversation

@Upd4ting

@Upd4ting Upd4ting commented Nov 13, 2026 •

Copy link
Copy Markdown
Member

Why

Run dms-api's CI on Windows in parallel with Linux, so Windows-only breakage shows up before a release (ANT-2187).

What

  • ci.yml passes windows: true to ci-node.yml@v1. The node and test jobs now also run on windows-latest. There was no hand-written Windows job to replace.
  • .gitattributes sets * text=auto eol=lf, so a checkout with core.autocrlf=true keeps LF line endings.
  • @antelopejs/tooling-configs goes from ^0.5.1 to ^0.5.8, which ships Windows support for the pack's checks. antelopejs-sync added allowBuilds: lefthook: false to pnpm-workspace.yaml.
  • frontend-vue/tests/backend-boundary.test.ts expects the paths in the form Vite reports on Windows. Vite names the importer with / separators. The scratch root is now resolved with realpathSync.native, which expands the 8.3 short name (RUNNER~1) that realpathSync keeps, so it matches the long path the resolver returns. On Linux both changes give the same strings as before. The plugin is unchanged.

No Windows-only skip, no lint rule change, no guardrails ceiling change.

Proof

Before, first run with windows: true (e167530): https://github.com/AntelopeJS/dms-api/actions/runs/43442373110. ci / Test .nvmrc on Windows failed:

FAIL tests/backend-boundary.test.ts > the backend boundary of the frontend build > fails on an import of the backend source, naming both files
Tests  1 failed | 179 passed (180)
AssertionError: expected [Function] to throw error including 'C:\Users\RUNNER~1\AppData\Local\Temp\…' but got '[commonjs--resolver] [plugi…'
Expected: "C:\Users\RUNNER~1\AppData\Local\Temp\dms-api-boundary-msZ06f\frontend-vue\entry.ts"
Received: "… Backend boundary: C:/Users/RUNNER~1/AppData/Local/Temp/dms-api-boundary-msZ06f/frontend-vue/entry.ts imports C:\Users\runneradmin\AppData\Local\Temp\dms-api-boundary-msZ06f\src\backend.ts, which is under the src of @antelopejs/dms-api. …"

After (78700e4): https://github.com/AntelopeJS/dms-api/actions/runs/43442902085. Every job passed, including ci / Node .nvmrc on Windows and ci / Test .nvmrc on Windows:

✓ tests/backend-boundary.test.ts (8 tests)
Test Files  17 passed (17)

Local, Linux: pnpm run checks prints Checks: 12 passed, 0 failed.

@Upd4ting
Upd4ting merged commit f8822df into main Nov 13, 2026
13 checks passed
@Upd4ting
Upd4ting deleted the ci/windows branch November 13, 2026 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant