Skip to content

Conversation

@jakex7
Copy link
Member

@jakex7 jakex7 commented Nov 19, 2024

Summary

Replace the deprecated TurboReactPackage with the newer BaseReactPackage to improve compatibility with future React Native releases. Note that this change requires at least [email protected]+.

Similar change has been introduced in the SVG package software-mansion/react-native-svg#2541

Test plan

CI should pass

@jakex7 jakex7 requested a review from j-piasecki November 19, 2024 12:53
@jakex7 jakex7 requested a review from j-piasecki November 20, 2024 14:31
@j-piasecki j-piasecki merged commit bf55fde into main Nov 29, 2024
3 checks passed
@j-piasecki j-piasecki deleted the @jakex7/useNewApi branch November 29, 2024 15:40
@RonRadtke
Copy link

RonRadtke commented Dec 17, 2024

Just since I stumbled over it after releasing the same changes for my lib: react-native-community/cli#2563

@jakex7 Right now the fixed versions of the cli are not shipped yet with new RN packages, and old packages without updated cli are broken too....

@jakex7
Copy link
Member Author

jakex7 commented Dec 18, 2024

Thanks for the info @RonRadtke! Fortunately RNGH should be safe as it falls into first check which is less strict and has been there for a long time
https://github.com/react-native-community/cli/blob/feba14cf36de2fdefa41c7052841a5ee34c19084/packages/cli-config-android/src/config/findPackageClassName.ts#L71

FYI @j-piasecki

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.

6 participants