Skip to content

Skip prebuilds for DynamicFrameworks CI jobs#56593

Merged
cipolleschi merged 2 commits into0.83-stablefrom
fix/dynamic-frameworks-no-prebuilds-083
Apr 24, 2026
Merged

Skip prebuilds for DynamicFrameworks CI jobs#56593
cipolleschi merged 2 commits into0.83-stablefrom
fix/dynamic-frameworks-no-prebuilds-083

Conversation

@cipolleschi
Copy link
Copy Markdown
Contributor

Summary

  • DynamicFrameworks CI jobs on 0.83-stable were incorrectly downloading and using prebuilds (ReactNativeDependencies + ReactCore xcframeworks), which could mask CocoaPods misconfigurations when running with use_frameworks.
  • Both test-ios-rntester and test-ios-helloworld composite actions now conditionally skip prebuild downloads and env vars when use-frameworks is set to DynamicFrameworks, ensuring these jobs always build from source.
  • This aligns the behavior with how the same jobs are configured on main.

Changelog:

[INTERNAL] - Skip prebuilds for DynamicFrameworks CI jobs to always build from source

Test plan

CI should pass — StaticLibraries jobs still use prebuilds as before; DynamicFrameworks jobs will now build from source.

DynamicFrameworks jobs should always build from source to catch
CocoaPods misconfigurations with use_frameworks. Previously, prebuilds
were always downloaded and used, masking potential failures.
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 24, 2026
@github-actions
Copy link
Copy Markdown

Warning

JavaScript API change detected

This PR commits an update to ReactNativeApi.d.ts, indicating a change to React Native's public JavaScript API.

  • Please include a clear changelog message.
  • This change will be subject to additional review.

This change was flagged as: BREAKING

The use-hermes-nightly flag was computed from the branch name, which
evaluates incorrectly for PR branches targeting stable. Instead, read
the actual HERMES_VERSION_NAME from version.properties and only use
nightly when it is the 1000.0.0 development placeholder.
@cipolleschi cipolleschi merged commit 3770421 into 0.83-stable Apr 24, 2026
65 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner Pick Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant