ci: do not expect bash on Windows

testdouble / teenytest

A very simple, zero-config test runner for Node.js

MIT License

96 stars 14 forks source link

ci: do not expect bash on Windows #81

Closed webstech closed 1 year ago

webstech commented 1 year ago

A full bash environment is only available with wsl2 so do not rely on it. Run spawned processes in cmd shell on Windows. package.json used for these tests now specify node should be run instead of relying on shebang support.

wsl2 is not available on the windows-latest build. Only git bash is available and it is still a win32 platform. As a result, in the spawned processes, npm still tries to run a cmd shell instead of a bash shell for running scripts.

searls commented 1 year ago

This was a real journey! I'll cut a release now so you can update your upstream PRs on quibble and testdouble

searls commented 1 year ago

Landed in v6.0.5