Skip to content

Commit 7305814

Browse files
pavelzemanclaude
andauthored
ci: add TypeScript and unit test jobs, fix pre-existing CI failures (#158)
* ci: add TypeScript and unit test jobs, fix pre-existing CI failures Builds on #161 (which added the base CI workflow with lint, Kotlin tests, and build). This commit adds the remaining two jobs: - TypeScript — npm run typescript (tsc --noEmit) - Unit Tests — npm test -- --coverage --passWithNoTests Fixes: - TypeScript: exclude example/ from tsconfig.json (references React Navigation deps only in example/node_modules) - Unit Tests: --passWithNoTests since repo has no JS/TS test files (all tests are Kotlin in test-runner/) Build job now depends on all four check jobs. Co-authored-by: Claude <claude@anthropic.com> * 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> --------- Co-authored-by: Claude <claude@anthropic.com>
1 parent be32ac8 commit 7305814

File tree

2 files changed

+14
-3
lines changed

2 files changed

+14
-3
lines changed

.github/workflows/ci.yml

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,18 @@ jobs:
2626
- run: npm ci
2727
- run: npm run lint
2828

29+
typescript:
30+
name: TypeScript
31+
runs-on: ubuntu-latest
32+
steps:
33+
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
34+
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
35+
with:
36+
node-version: 22
37+
cache: npm
38+
- run: npm ci
39+
- run: npm run typescript
40+
2941
kotlin-tests:
3042
name: Kotlin Tests
3143
runs-on: ubuntu-latest
@@ -43,7 +55,7 @@ jobs:
4355
build:
4456
name: Build Package
4557
runs-on: ubuntu-latest
46-
needs: [lint, kotlin-tests]
58+
needs: [lint, typescript, kotlin-tests]
4759
steps:
4860
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
4961
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4

tsconfig.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@
3434
"babel.config.js",
3535
"metro.config.js",
3636
"jest.config.js",
37-
"example/detox/**/*",
38-
"example/node_modules"
37+
"example"
3938
]
4039
}

0 commit comments

Comments
 (0)