Skip to content

Conversation

@lposen
Copy link
Contributor

@lposen lposen commented Jun 11, 2025

🔹 JIRA Ticket(s) if any

✏️ Description

The new versions of RN suggest that you use .mm files (Objective-C++) instead of .m files (Objective-C). This implements that by changing RNIterableAPI.m to RNIterableAPI.mm in the main ios sdk, and changing AppDelegate.m to AppDelegate.mm in the example ios app.

Testing

  • cd into the example directory
  • Run yarn install
  • cd into the ios directory, and run pod install
  • cd back into the example directory
  • run yarn start
  • in a separate terminal, run yarn ios

The app should build and run correctly

@github-actions
Copy link

github-actions bot commented Jun 11, 2025

Lines Statements Branches Functions
Coverage: 38%
38.11% (178/467) 11.29% (20/177) 32.46% (50/154)

@lposen lposen marked this pull request as ready for review June 25, 2025 16:54
@lposen lposen merged commit 2e67a25 into new-arch/master Jul 2, 2025
5 checks passed
@lposen lposen deleted the new-arch/mob-11603 branch July 2, 2025 20:15
@lposen lposen mentioned this pull request Oct 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants