Skip to content

Commit fcb20de

Browse files
committed
Build before linting
1 parent cc7ffb4 commit fcb20de

File tree

1 file changed

+17
-0
lines changed

1 file changed

+17
-0
lines changed

.github/workflows/check.yml

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,24 @@ 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 }}
38+
- run: rustup target add x86_64-linux-android
2739
- run: npm ci
40+
- run: npm run build
41+
# Bootstrap host package to get weak-node-api and ferric-example to get types
42+
# 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
43+
- run: npm run bootstrap --workspace react-native-node-api
44+
- run: npm run bootstrap --workspace @react-native-node-api/ferric-example
2845
- run: npm run lint
2946
env:
3047
DEBUG: eslint:eslint

0 commit comments

Comments
 (0)