Skip to content

Commit 532e86f

Browse files
committed
Make the HelloWorld dynamic-frameworks CI lane use dynamic frameworks
#54482 switched the test_ios_helloworld matrix to use_frameworks: [false, true] but left the test-ios-helloworld action comparing the input to "DynamicFrameworks", so the true lane builds with static libraries and the Debug exclude (StaticLibraries) matches nothing. Compare against "true" like test-ios-rntester does and exclude use_frameworks: false.
1 parent 8d658f4 commit 532e86f

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@ name: test-ios-helloworld
22
description: Test iOS Hello World
33
inputs:
44
use-frameworks:
5-
description: The dependency building and linking strategy to use. Must be one of "StaticLibraries", "DynamicFrameworks"
6-
default: StaticLibraries
5+
description: Whether to build with dynamic frameworks.
6+
default: false
77
ruby-version:
88
description: The version of ruby that must be used
99
default: 2.6.10
@@ -51,7 +51,7 @@ runs:
5151
cd private/helloworld
5252
args=()
5353
54-
if [[ ${{ inputs.use-frameworks }} == "DynamicFrameworks" ]]; then
54+
if [[ "${{ inputs.use-frameworks }}" == "true" ]]; then
5555
args+=(--frameworks dynamic)
5656
fi
5757

‎.github/workflows/test-all.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -534,7 +534,7 @@ jobs:
534534
exclude:
535535
# This config is tested with Ruby 3.2.0. Let's not double test it.
536536
- flavor: Debug
537-
use_frameworks: StaticLibraries
537+
use_frameworks: false
538538
env:
539539
PROJECT_NAME: iOSTemplateProject
540540
YARN_ENABLE_IMMUTABLE_INSTALLS: false

0 commit comments

Comments
 (0)