Skip to content

Generate public feature flags entry point (#58742) - #58742

Closed
j-piasecki wants to merge 1 commit into
react:mainfrom
j-piasecki:export-D122353811
Closed

j-piasecki wants to merge 1 commit into
react:mainfrom
j-piasecki:export-D122353811

Conversation

@j-piasecki

@j-piasecki j-piasecki commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Summary:

Adds a dangerouslyExposeInPublicCppHeaders option to the feature flags config. Flags that opt in are generated into ReactNativePublicFeatureFlags.h, whose getters forward to ReactNativeFeatureFlags from a generated .cpp, so exported headers can read them without including the rest of the module. Opts in the flags that exported headers currently read inline.

Changelog: [Internal]

Reviewed By: rubennorte

Differential Revision: D122353811

@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 Sep 29, 2026
@facebook-github-tools facebook-github-tools Bot added p: Software Mansion Partner: Software Mansion Partner p: Facebook Partner: Facebook labels Sep 29, 2026
@meta-codesync meta-codesync Bot changed the title Generate public feature flags entry point Generate public feature flags entry point (#58742) Sep 30, 2026
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Sep 30, 2026
Summary:

Adds a `dangerouslyExposeInPublicCppHeaders` option to the feature flags config. Flags that opt in are generated into `ReactNativePublicFeatureFlags.h`, whose getters forward to `ReactNativeFeatureFlags` from a generated `.cpp`, so exported headers can read them without including the rest of the module. Opts in the flags that exported headers currently read inline.

Changelog: [Internal]

Reviewed By: rubennorte

Differential Revision: D122353811
j-piasecki added a commit to j-piasecki/react-native that referenced this pull request Sep 30, 2026
Summary:
Pull Request resolved: react#58742

Adds a `dangerouslyExposeInPublicCppHeaders` option to the feature flags config. Flags that opt in are generated into `ReactNativePublicFeatureFlags.h`, whose getters forward to `ReactNativeFeatureFlags` from a generated `.cpp`, so exported headers can read them without including the rest of the module. Opts in the flags that exported headers currently read inline.

Changelog: [Internal]

Reviewed By: rubennorte

Differential Revision: D122353811
@meta-codesync

meta-codesync Bot commented Sep 30, 2026

Copy link
Copy Markdown

@j-piasecki has exported this pull request. If you are a Meta employee, you can view the originating Diff in D122353811.

Summary:
Pull Request resolved: react#58742

Adds a `dangerouslyExposeInPublicCppHeaders` option to the feature flags config. Flags that opt in are generated into `ReactNativePublicFeatureFlags.h`, whose getters forward to `ReactNativeFeatureFlags` from a generated `.cpp`, so exported headers can read them without including the rest of the module. Opts in the flags that exported headers currently read inline.

Changelog: [Internal]

Reviewed By: rubennorte

Differential Revision: D122353811
@meta-codesync meta-codesync Bot closed this in 6a6e599 Sep 30, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Sep 30, 2026
@meta-codesync

meta-codesync Bot commented Sep 30, 2026

Copy link
Copy Markdown

This pull request has been merged in 6a6e599.

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. Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook p: Software Mansion Partner: Software Mansion Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant