Skip to content

Commit eb43025

Browse files
committed
fix: add separate build job to satisfy GitHub branch protection requirements
- Split build steps from test job into dedicated build job - Add dependency (needs: test) to ensure tests pass before building - Maintain matrix strategy for both Node.js 18.x and 20.x - Keep CLI functionality testing in build job This resolves the 'build' status check that was stuck on 'Expected Waiting for status to be reported' in GitHub PR requirements.
1 parent 8b1b498 commit eb43025

File tree

1 file changed

+21
-0
lines changed

1 file changed

+21
-0
lines changed

.github/workflows/ci.yml

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,27 @@ jobs:
3535

3636
- name: Run tests
3737
run: npm test
38+
39+
build:
40+
runs-on: ubuntu-latest
41+
needs: test
42+
43+
strategy:
44+
matrix:
45+
node-version: [18.x, 20.x]
46+
47+
steps:
48+
- name: Checkout code
49+
uses: actions/checkout@v4
50+
51+
- name: Setup Node.js ${{ matrix.node-version }}
52+
uses: actions/setup-node@v4
53+
with:
54+
node-version: ${{ matrix.node-version }}
55+
cache: 'npm'
56+
57+
- name: Install dependencies
58+
run: npm ci
3859

3960
- name: Build CLI
4061
run: npm run build

0 commit comments

Comments
 (0)