-
-
Notifications
You must be signed in to change notification settings - Fork 537
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Importing of project into Swift projects #2160
fix: Importing of project into Swift projects #2160
Conversation
Hi @Brett-Best, thanks for taking your time to introduce these changes 🎉 We will test these changes in |
Thanks, it might be worth including |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems we're fine on front of reanimated interop & also seems that it works well in runtime. Thanks!
## Description Fixes software-mansion#2158 by making C++ headers project instead of public. Original code sample (in the issue) can be updated to include this change verifying the fix. ## Checklist - [x] Included code example that can be used to test this change - [ ] Updated TS types - [ ] Updated documentation: <!-- For adding new props to native-stack --> - [ ] https://github.com/software-mansion/react-native-screens/blob/main/guides/GUIDE_FOR_LIBRARY_AUTHORS.md - [ ] https://github.com/software-mansion/react-native-screens/blob/main/native-stack/README.md - [ ] https://github.com/software-mansion/react-native-screens/blob/main/src/types.tsx - [ ] https://github.com/software-mansion/react-native-screens/blob/main/src/native-stack/types.tsx - [ ] Ensured that CI passes
Description
Fixes #2158 by making C++ headers project instead of public.
Original code sample (in the issue) can be updated to include this change verifying the fix.
Checklist