Skip to content

Commit 7741ad4

Browse files
author
Taras Mankovski
committed
Consolidate CI workflows into single test.yaml
- Combine verify-posix, verify-windows, and effection-compat into test.yaml - Run lint, test-posix, and test-windows jobs in parallel - Windows tests use CMD shell (fixes graceful shutdown issues with bash) - Each test job runs both unit tests and effection compatibility tests - Update publish.yaml to depend on consolidated test workflow
1 parent 47e71eb commit 7741ad4

4 files changed

Lines changed: 36 additions & 100 deletions

File tree

.github/workflows/effection-compat.yaml

Lines changed: 0 additions & 38 deletions
This file was deleted.

.github/workflows/publish.yaml

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -6,17 +6,11 @@ on:
66
- main
77

88
jobs:
9-
verify-posix:
10-
uses: ./.github/workflows/verify-posix.yaml
11-
12-
verify-windows:
13-
uses: ./.github/workflows/verify-windows.yaml
14-
15-
compat:
16-
uses: ./.github/workflows/effection-compat.yaml
9+
test:
10+
uses: ./.github/workflows/test.yaml
1711

1812
generate-matrix:
19-
needs: [verify-posix, verify-windows, compat]
13+
needs: [test]
2014
name: Generate Job Matrix
2115
runs-on: ubuntu-latest
2216
outputs:
Lines changed: 33 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
1-
name: Verify (POSIX)
1+
name: Test
22

33
on:
44
workflow_call:
55
pull_request:
6-
branches:
7-
- main
6+
branches: [main]
7+
workflow_dispatch:
88

99
permissions:
1010
contents: read
@@ -37,14 +37,40 @@ jobs:
3737
- name: Check tsconfig refs
3838
run: pnpm check:tsrefs
3939

40-
test:
40+
test-posix:
4141
name: Test (${{ matrix.os }})
4242
strategy:
4343
fail-fast: false
4444
matrix:
4545
os: [ubuntu-latest, macos-latest]
4646
runs-on: ${{ matrix.os }}
47+
steps:
48+
- uses: actions/checkout@v4
49+
50+
- uses: pnpm/action-setup@v4
51+
52+
- uses: actions/setup-node@v4
53+
with:
54+
node-version: 22
55+
cache: pnpm
4756

57+
- run: pnpm install --frozen-lockfile
58+
59+
- name: Build
60+
run: pnpm build
61+
62+
- name: Run tests
63+
run: pnpm test
64+
65+
- name: Run effection compatibility tests
66+
run: pnpm test:effection
67+
68+
test-windows:
69+
name: Test (windows)
70+
runs-on: windows-latest
71+
defaults:
72+
run:
73+
shell: cmd
4874
steps:
4975
- uses: actions/checkout@v4
5076

@@ -62,3 +88,6 @@ jobs:
6288

6389
- name: Run tests
6490
run: pnpm test
91+
92+
- name: Run effection compatibility tests
93+
run: pnpm test:effection

.github/workflows/verify-windows.yaml

Lines changed: 0 additions & 49 deletions
This file was deleted.

0 commit comments

Comments
 (0)