Skip to content

Commit fdec9e6

Browse files
xThreehmeta-codesync[bot]
authored andcommitted
Make the HelloWorld dynamic-frameworks CI lane actually use dynamic frameworks (#58962)
Summary: #54482 changed the `test_ios_helloworld` matrix from `use_frameworks: [StaticLibraries, DynamicFrameworks]` to `[false, true]` and updated `test-ios-rntester` to match, but `test-ios-helloworld` still checks the input against `"DynamicFrameworks"`: ```bash if [[ ${{ inputs.use-frameworks }} == "DynamicFrameworks" ]]; then args+=(--frameworks dynamic) fi ``` So the `true` lane builds with static libraries, the same as the `false` lane, and HelloWorld with dynamic frameworks is no longer tested. The Debug exclude (`use_frameworks: StaticLibraries`) also matches nothing, so `(Debug, false)` runs even though the Ruby 3.2.0 job already covers it. The latest Test All run on main shows all four `test_ios_helloworld` combinations. This makes the action compare the input to `"true"`, as `test-ios-rntester` does, and changes the exclude to `use_frameworks: false`. The Ruby 3.2.0 job passes no `use-frameworks`, so it keeps the default and still builds static libraries. ## Changelog: [INTERNAL] [FIXED] - Make the HelloWorld dynamic-frameworks CI lane use dynamic frameworks Pull Request resolved: #58962 Test Plan: - `actionlint` on `test-all.yml`: the "value StaticLibraries in exclude does not match" error is gone. - On this PR, Test All should run three `test_ios_helloworld` jobs, and the `(…, true)` ones should pass `--frameworks dynamic` to `yarn bootstrap ios`. Reviewed By: christophpurrer Differential Revision: D124991517 Pulled By: cipolleschi fbshipit-source-id: 0fc93840150ef014feee2499183cb1f1605d0bbf
1 parent dcc38df commit fdec9e6

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)