Skip to content

Fix minor issues in the Turbo Modules iOS doc#4892

Open
sepeth wants to merge 1 commit intofacebook:mainfrom
sepeth:Minor_fixes_in_turbo_modules_doc
Open

Fix minor issues in the Turbo Modules iOS doc#4892
sepeth wants to merge 1 commit intofacebook:mainfrom
sepeth:Minor_fixes_in_turbo_modules_doc

Conversation

@sepeth
Copy link

@sepeth sepeth commented Nov 19, 2025

Hi there,

Main issue was that instead of open TurboModuleExample.xcworkspace, open SampleApp.xcworkspace was used.

NS_ASSUME_NONNULL_END should already be existing in the file, but I made the mistake of clicking the copy button at the top when viewing the docs, and pasting directly into the file. Given that it is just one more line, it wouldn't hurt to have it in the diff.

And, I have a question, I see some copies of this in the website/versioned_docs/**/* as well, should I update those as well, or is there a script/automation that does this?

@meta-cla meta-cla bot added the CLA Signed label Nov 19, 2025
@netlify
Copy link

netlify bot commented Nov 19, 2025

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit 659ae6f
🔍 Latest deploy log https://app.netlify.com/projects/react-native/deploys/691db5359d1a5f00080edf74
😎 Deploy Preview https://deploy-preview-4892--react-native.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments