Skip to content

RCTTurboModule nits - #54692

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

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

Conversation

@javache

@javache javache commented Nov 25, 2025

Copy link
Copy Markdown
Contributor

Summary:
Lazily create error strings and use jsi::JSError for more user-friendly error messages

Changelog: [Internal]

Differential Revision: D87868701

@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 25, 2025
@meta-codesync

meta-codesync Bot commented Nov 25, 2025

Copy link
Copy Markdown

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

Summary:

Lazily create error strings and use jsi::JSError for more user-friendly error messages

Changelog: [Internal]

Reviewed By: cipolleschi

Differential Revision: D87868701
@meta-codesync meta-codesync Bot closed this in d49e07a Dec 1, 2025
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Dec 1, 2025
@meta-codesync

meta-codesync Bot commented Dec 1, 2025

Copy link
Copy Markdown

This pull request has been merged in d49e07a.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants