Skip to content

Commit 5841e2d

Browse files
Copilotkraenhansen
andcommitted
Fix workflow permissions and remove unnecessary dependencies for visionOS workflow
Co-authored-by: kraenhansen <1243959+kraenhansen@users.noreply.github.com>
1 parent 35a812d commit 5841e2d

File tree

1 file changed

+5
-18
lines changed

1 file changed

+5
-18
lines changed

.github/workflows/check-visionos.yml

Lines changed: 5 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,5 @@
11
name: Check visionOS
22

3-
env:
4-
# Version here should match the one in React Native template and packages/cmake-rn/src/cli.ts
5-
NDK_VERSION: 27.1.12297006
6-
# Enabling the Gradle test on CI (disabled by default because it downloads a lot)
7-
ENABLE_GRADLE_TESTS: true
8-
93
on:
104
push:
115
branches:
@@ -21,24 +15,17 @@ jobs:
2115
test-visionos:
2216
name: Test Ferric with visionOS Triplets
2317
runs-on: macos-latest
18+
permissions:
19+
contents: read
2420
steps:
2521
- uses: actions/checkout@v4
2622
- uses: actions/setup-node@v4
2723
with:
28-
node-version: lts/jod
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 }}
24+
node-version: lts/iron
3825
# Install visionOS targets for Rust
3926
- run: rustup target add aarch64-apple-visionos aarch64-apple-visionos-sim
40-
# Also install other required targets
41-
- run: rustup target add x86_64-linux-android aarch64-linux-android armv7-linux-androideabi i686-linux-android aarch64-apple-ios-sim
27+
# Also install required iOS targets for the bootstrap process
28+
- run: rustup target add aarch64-apple-ios-sim
4229
- run: npm ci
4330
- run: npm run build
4431
# Bootstrap host package to get weak-node-api and ferric-example with visionOS support

0 commit comments

Comments
 (0)