Skip to content

Commit 9207eef

Browse files
committed
Skip prebuilds for DynamicFrameworks CI jobs
DynamicFrameworks jobs should always build from source to catch CocoaPods misconfigurations with use_frameworks. Previously, prebuilds were always downloaded and used, masking potential failures.
1 parent e745903 commit 9207eef

2 files changed

Lines changed: 14 additions & 6 deletions

File tree

.github/actions/test-ios-helloworld/action.yml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,19 +38,23 @@ runs:
3838
if: ${{ inputs.use-hermes-nightly == 'true' }}
3939
uses: ./.github/actions/yarn-install
4040
- name: Download ReactNativeDependencies
41+
if: ${{ inputs.use-frameworks != 'DynamicFrameworks' }}
4142
uses: actions/download-artifact@v4
4243
with:
4344
name: ReactNativeDependencies${{ inputs.flavor }}.xcframework.tar.gz
4445
path: /tmp/third-party
4546
- name: Print third-party folder
47+
if: ${{ inputs.use-frameworks != 'DynamicFrameworks' }}
4648
shell: bash
4749
run: ls -lR /tmp/third-party
4850
- name: Download React Native Prebuilds
51+
if: ${{ inputs.use-frameworks != 'DynamicFrameworks' }}
4952
uses: actions/download-artifact@v4
5053
with:
5154
name: ReactCore${{ inputs.flavor }}.xcframework.tar.gz
5255
path: /tmp/ReactCore
5356
- name: Print ReactCore folder
57+
if: ${{ inputs.use-frameworks != 'DynamicFrameworks' }}
5458
shell: bash
5559
run: ls -lR /tmp/ReactCore
5660
- name: Install iOS dependencies - Configuration ${{ inputs.flavor }};
@@ -61,11 +65,11 @@ runs:
6165
6266
if [[ ${{ inputs.use-frameworks }} == "DynamicFrameworks" ]]; then
6367
args+=(--frameworks dynamic)
68+
else
69+
export RCT_USE_LOCAL_RN_DEP="/tmp/third-party/ReactNativeDependencies${{ inputs.flavor }}.xcframework.tar.gz"
70+
export RCT_TESTONLY_RNCORE_TARBALL_PATH="/tmp/ReactCore/ReactCore${{ inputs.flavor }}.xcframework.tar.gz"
6471
fi
6572
66-
export RCT_USE_LOCAL_RN_DEP="/tmp/third-party/ReactNativeDependencies${{ inputs.flavor }}.xcframework.tar.gz"
67-
export RCT_TESTONLY_RNCORE_TARBALL_PATH="/tmp/ReactCore/ReactCore${{ inputs.flavor }}.xcframework.tar.gz"
68-
6973
yarn bootstrap ios "${args[@]}" | cat
7074
7175
- name: Run Helloworld tests

.github/actions/test-ios-rntester/action.yml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,29 +49,33 @@ runs:
4949
if: ${{ inputs.run-unit-tests == 'true' }}
5050
uses: ./.github/actions/prepare-ios-tests
5151
- name: Download ReactNativeDependencies
52+
if: ${{ inputs.use-frameworks != 'DynamicFrameworks' }}
5253
uses: actions/download-artifact@v4
5354
with:
5455
name: ReactNativeDependencies${{ inputs.flavor }}.xcframework.tar.gz
5556
path: /tmp/third-party/
5657
- name: Print third-party folder
58+
if: ${{ inputs.use-frameworks != 'DynamicFrameworks' }}
5759
shell: bash
5860
run: ls -lR /tmp/third-party
5961
- name: Download React Native Prebuilds
62+
if: ${{ inputs.use-frameworks != 'DynamicFrameworks' }}
6063
uses: actions/download-artifact@v4
6164
with:
6265
name: ReactCore${{ inputs.flavor }}.xcframework.tar.gz
6366
path: /tmp/ReactCore
6467
- name: Print ReactCore folder
68+
if: ${{ inputs.use-frameworks != 'DynamicFrameworks' }}
6569
shell: bash
6670
run: ls -lR /tmp/ReactCore
6771
- name: Install CocoaPods dependencies
6872
shell: bash
6973
run: |
70-
export RCT_USE_LOCAL_RN_DEP="/tmp/third-party/ReactNativeDependencies${{ inputs.flavor }}.xcframework.tar.gz"
71-
export RCT_TESTONLY_RNCORE_TARBALL_PATH="/tmp/ReactCore/ReactCore${{ inputs.flavor }}.xcframework.tar.gz"
72-
7374
if [[ ${{ inputs.use-frameworks }} == "DynamicFrameworks" ]]; then
7475
export USE_FRAMEWORKS=dynamic
76+
else
77+
export RCT_USE_LOCAL_RN_DEP="/tmp/third-party/ReactNativeDependencies${{ inputs.flavor }}.xcframework.tar.gz"
78+
export RCT_TESTONLY_RNCORE_TARBALL_PATH="/tmp/ReactCore/ReactCore${{ inputs.flavor }}.xcframework.tar.gz"
7579
fi
7680
7781
cd packages/rn-tester

0 commit comments

Comments
 (0)