Skip to content

Commit c4d4515

Browse files
pavelzemanclaude
andcommitted
ci: remove no-op Jest unit-tests job
There are zero JS/TS test files in this repo, so the Jest job was burning ~60-90s of CI time per run for nothing. Removed per reviewer feedback — can re-add when JS/TS tests are introduced. Co-authored-by: Claude <claude@anthropic.com>
1 parent 88feb18 commit c4d4515

1 file changed

Lines changed: 1 addition & 13 deletions

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -38,18 +38,6 @@ jobs:
3838
- run: npm ci
3939
- run: npm run typescript
4040

41-
unit-tests:
42-
name: Unit Tests
43-
runs-on: ubuntu-latest
44-
steps:
45-
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
46-
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
47-
with:
48-
node-version: 22
49-
cache: npm
50-
- run: npm ci
51-
- run: npm test -- --coverage --passWithNoTests
52-
5341
kotlin-tests:
5442
name: Kotlin Tests
5543
runs-on: ubuntu-latest
@@ -67,7 +55,7 @@ jobs:
6755
build:
6856
name: Build Package
6957
runs-on: ubuntu-latest
70-
needs: [lint, typescript, unit-tests, kotlin-tests]
58+
needs: [lint, typescript, kotlin-tests]
7159
steps:
7260
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
7361
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4

0 commit comments

Comments
 (0)