Skip to content

Commit 300852d

Browse files
committed
ci: use bash shell for tests in windows
1 parent fc26dd8 commit 300852d

File tree

4 files changed

+6
-3
lines changed

4 files changed

+6
-3
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ jobs:
3131
npm install --ignore-scripts
3232
3333
- name: Run tests
34+
shell: bash
3435
run: |
3536
npm run test:ci
3637

.github/workflows/package-manager-ci.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ jobs:
2929
- name: Install dependancies
3030
run: pnpm install
3131
- name: Tests
32+
shell: bash
3233
run: pnpm run test:ci
3334

3435
yarn-pnp:
@@ -56,4 +57,5 @@ jobs:
5657
# needed due the yarn.lock file in repository's .gitignore
5758
YARN_ENABLE_IMMUTABLE_INSTALLS: false
5859
- name: Tests
60+
shell: bash
5961
run: yarn run test:yarn

lib/worker.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ async function start () {
2525
} else if (process.env.TS_NODE_DEV) {
2626
realRequire('ts-node-dev')
2727
}
28-
// TODO: Add proper support for ES modules once TS supports them properly
28+
// TODO:Support ES imports once tsc & ts-node provide better compatibility guarantees.
2929
fn = realRequire(decodeURIComponent(filename.replace('file://', '')))
3030
} else {
3131
fn = (await realImport(filename))

package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
"sonic-boom": "^2.0.1",
1616
"standard": "^16.0.3",
1717
"tap": "^16.0.0",
18-
"ts-node": "^10.0.0",
18+
"ts-node": "^10.7.0",
1919
"typescript": "^4.6.0",
2020
"why-is-node-running": "^2.2.0"
2121
},
@@ -24,7 +24,7 @@
2424
"test:ci": "standard && npm run transpile && npm run test:ci:js && npm run test:ci:ts",
2525
"test:ci:js": "tap --no-check-coverage --coverage-report=lcovonly \"test/**/*.test.*js\"",
2626
"test:ci:ts": "tap --ts --no-check-coverage --coverage-report=lcovonly \"test/**/*.test.*ts\"",
27-
"test:yarn": "tap \"test/**/*.test.js\" --no-check-coverage",
27+
"test:yarn": "npm run transpile && tap \"test/**/*.test.js\" --no-check-coverage",
2828
"transpile": "./test/ts/transpile.sh",
2929
"prepare": "husky install"
3030
},

0 commit comments

Comments
 (0)