Skip to content

Allow the existing StatusBar getter’s synchronous main-queue call (#59998) - #59998

Open
javache wants to merge 1 commit into
react:mainfrom
javache:export-D131753903
Open

javache wants to merge 1 commit into
react:mainfrom
javache:export-D131753903

Conversation

@javache

@javache javache commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

Summary:

Keep the existing getVerticalBarEdge implementation and allow its Clang-Tidy synchronous-main-queue diagnostic at this call site only. No runtime behavior or public API changes.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D131753903

@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 Nov 13, 2026
@meta-codesync

meta-codesync Bot commented Nov 13, 2026

Copy link
Copy Markdown

@javache has exported this pull request. If you are a Meta employee, you can view the originating Diff in D131753903.

@cortinico cortinico left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review automatically exported from Phabricator review in Meta.

@github-actions

Copy link
Copy Markdown

Visual regression

No visual differences at be47f9bc9d08.

android: no capture

@meta-codesync meta-codesync Bot changed the title Allow the existing StatusBar getter’s synchronous main-queue call Allow the existing StatusBar getter’s synchronous main-queue call (#59998) Nov 13, 2026
javache added a commit to javache/react-native that referenced this pull request Nov 13, 2026
…act#59998)

Summary:

Keep the existing `getVerticalBarEdge` implementation and allow its Clang-Tidy synchronous-main-queue diagnostic at this call site only. No runtime behavior or public API changes.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D131753903
…act#59998)

Summary:

Keep the existing `getVerticalBarEdge` implementation and allow its Clang-Tidy synchronous-main-queue diagnostic at this call site only. No runtime behavior or public API changes.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D131753903

This branch has not been deployed

No deployments
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. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants