Skip to content

Commit

Permalink
ci: use bash shell for tests in windows
Browse files Browse the repository at this point in the history
  • Loading branch information
castarco committed Mar 21, 2022
1 parent fc26dd8 commit 300852d
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 3 deletions.
1 change: 1 addition & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ jobs:
npm install --ignore-scripts
- name: Run tests
shell: bash
run: |
npm run test:ci
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/package-manager-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ jobs:
- name: Install dependancies
run: pnpm install
- name: Tests
shell: bash
run: pnpm run test:ci

yarn-pnp:
Expand Down Expand Up @@ -56,4 +57,5 @@ jobs:
# needed due the yarn.lock file in repository's .gitignore
YARN_ENABLE_IMMUTABLE_INSTALLS: false
- name: Tests
shell: bash
run: yarn run test:yarn
2 changes: 1 addition & 1 deletion lib/worker.js
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ async function start () {
} else if (process.env.TS_NODE_DEV) {
realRequire('ts-node-dev')
}
// TODO: Add proper support for ES modules once TS supports them properly
// TODO:Support ES imports once tsc & ts-node provide better compatibility guarantees.
fn = realRequire(decodeURIComponent(filename.replace('file://', '')))
} else {
fn = (await realImport(filename))
Expand Down
4 changes: 2 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@
"sonic-boom": "^2.0.1",
"standard": "^16.0.3",
"tap": "^16.0.0",
"ts-node": "^10.0.0",
"ts-node": "^10.7.0",
"typescript": "^4.6.0",
"why-is-node-running": "^2.2.0"
},
Expand All @@ -24,7 +24,7 @@
"test:ci": "standard && npm run transpile && npm run test:ci:js && npm run test:ci:ts",
"test:ci:js": "tap --no-check-coverage --coverage-report=lcovonly \"test/**/*.test.*js\"",
"test:ci:ts": "tap --ts --no-check-coverage --coverage-report=lcovonly \"test/**/*.test.*ts\"",
"test:yarn": "tap \"test/**/*.test.js\" --no-check-coverage",
"test:yarn": "npm run transpile && tap \"test/**/*.test.js\" --no-check-coverage",
"transpile": "./test/ts/transpile.sh",
"prepare": "husky install"
},
Expand Down

0 comments on commit 300852d

Please sign in to comment.