Skip to content

Commit 0923e68

Browse files
committed
Build before linting
1 parent cc7ffb4 commit 0923e68

File tree

1 file changed

+16
-0
lines changed

1 file changed

+16
-0
lines changed

.github/workflows/check.yml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,23 @@ jobs:
2424
- uses: actions/setup-node@v4
2525
with:
2626
node-version: lts/jod
27+
# Set up JDK and Android SDK only because we need weak-node-api, to build ferric-example and to run the linting
28+
# TODO: Remove this once we have a way to run linting without building the native code
29+
- name: Set up JDK 17
30+
uses: actions/setup-java@v3
31+
with:
32+
java-version: "17"
33+
distribution: "temurin"
34+
- name: Setup Android SDK
35+
uses: android-actions/setup-android@v3
36+
with:
37+
packages: tools platform-tools ndk;${{ env.NDK_VERSION }}
2738
- run: npm ci
39+
- run: npm run build
40+
# Bootstrap host package to get weak-node-api and ferric-example to get types
41+
# TODO: Solve this by adding an option to ferric to build only types or by committing the types into the repo as a fixture for an "init" command
42+
- run: npm run bootstrap --workspace react-native-node-api
43+
- run: npm run bootstrap --workspace ferric-example
2844
- run: npm run lint
2945
env:
3046
DEBUG: eslint:eslint

0 commit comments

Comments
 (0)