Skip to content

Commit 940eb58

Browse files
authored
refactor(ci): remove redundant build job (#180)
The test job already runs `npm run build` for all Node.js versions in the matrix, including the latest version which matches .nvmrc. The separate build job duplicated this work without adding value - artifact checks are unnecessary since build failures already fail the test job. This reduces CI time by eliminating redundant checkout, npm ci, and build steps.
1 parent 591da4e commit 940eb58

1 file changed

Lines changed: 0 additions & 33 deletions

File tree

.github/workflows/ci.yml

Lines changed: 0 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -119,36 +119,3 @@ jobs:
119119
fail_ci_if_error: false
120120
env:
121121
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
122-
123-
build:
124-
name: Build
125-
runs-on: ubuntu-latest
126-
needs: [setup, test]
127-
128-
steps:
129-
- name: Checkout code
130-
uses: actions/checkout@v6
131-
132-
- name: Setup Node.js from .nvmrc
133-
uses: actions/setup-node@v6
134-
with:
135-
node-version-file: '.nvmrc'
136-
cache: 'npm'
137-
138-
- name: Install dependencies
139-
run: npm ci
140-
141-
- name: Build all packages
142-
run: npm run build
143-
144-
- name: Check for build artifacts
145-
run: |
146-
if [ ! -d "packages/driver-types/dist" ]; then
147-
echo "Error: driver-types build failed"
148-
exit 1
149-
fi
150-
if [ ! -d "packages/driver-xymd1/dist" ]; then
151-
echo "Error: driver-xymd1 build failed"
152-
exit 1
153-
fi
154-
echo "All packages built successfully"

0 commit comments

Comments
 (0)