Skip to content

Commit

Permalink
Add windows and macos to CI build matrix
Browse files Browse the repository at this point in the history
  • Loading branch information
sehrope committed Jan 5, 2024
1 parent f1b4b2f commit 5102bc7
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 8 deletions.
15 changes: 13 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,7 @@ jobs:
run: make clean deps lint

test:
name: Test - Node v${{ matrix.node_version }}
runs-on: ubuntu-latest
name: Test - ${{ matrix.os }} x Node v${{ matrix.node_version }}
strategy:
fail-fast: false
matrix:
Expand All @@ -29,13 +28,25 @@ jobs:
- '20'
- 'lts/*'
- 'latest'
os: [ubuntu-latest, windows-latest, macos-latest]
runs-on: ${{ matrix.os }}
steps:
- uses: actions/checkout@v4
- name: Setup Node
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node_version }}
cache: npm
- name: Setup docker (MacOS)
if: runner.os == 'macos'
run: |
brew install docker
colima start
- name: Show env
run: |
node --version
uname -a
docker --version
- name: Compile
run: make clean deps compile
- name: Setup Postgres
Expand Down
12 changes: 6 additions & 6 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -9,28 +9,28 @@ clean:
rm -rf lib node_modules coverage

compile:
node_modules/.bin/tsc --declaration
./node_modules/.bin/tsc --declaration

watch:
node_modules/.bin/tsc --watch --declaration
./node_modules/.bin/tsc --watch --declaration

test:
node_modules/.bin/ts-mocha test/suite.ts --exit $(TEST_ARGS)
./node_modules/.bin/ts-mocha test/suite.ts --exit $(TEST_ARGS)

clean-cov:
rm -rf coverage

test-cov:
node_modules/.bin/c8 \
./node_modules/.bin/c8 \
--require source-map-support/register \
--require ts-node/register \
--reporter=html \
--reporter=lcov \
--extension .ts \
node_modules/.bin/ts-mocha --exit test/suite.ts
./node_modules/.bin/ts-mocha --exit test/suite.ts

lint:
node_modules/.bin/eslint src
./node_modules/.bin/eslint src

package: clean deps compile

Expand Down

0 comments on commit 5102bc7

Please sign in to comment.