Skip to content
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

Remove deprecated ReanimatedJSIModulePackage #3481

Merged

Conversation

tomekzaw
Copy link
Member

@tomekzaw tomekzaw commented Aug 17, 2022

Description

This PR removes ReanimatedJSIModulePackage class which was used for initializing Reanimated 2.4.1 and below.

Since 2.5.0, Reanimated automatically installs itself on Android.

You can remove getJSIModulePackage() method in MainApplication.java of your app.

Changes

  • Removed deprecated class ReanimatedJSIModulePackage

Test code and steps to reproduce

Checklist

  • Included code example that can be used to test this change
  • Updated TS types
  • Added TS types tests
  • Added unit / integration tests
  • Updated documentation
  • Ensured that CI passes

@tomekzaw tomekzaw requested a review from piaskowyk August 17, 2022 07:41
@tomekzaw tomekzaw changed the title Remove deprecated ReanimatedJSIModulePackage [v3] Remove deprecated ReanimatedJSIModulePackage Aug 17, 2022
@tomekzaw tomekzaw changed the title [v3] Remove deprecated ReanimatedJSIModulePackage Remove deprecated ReanimatedJSIModulePackage Aug 17, 2022
@piaskowyk piaskowyk merged commit c4e4089 into main Aug 23, 2022
@piaskowyk piaskowyk deleted the @tomekzaw/remove-deprecated-reanimatedjsimodulepackage branch August 23, 2022 08:23
fluiddot pushed a commit to wordpress-mobile/react-native-reanimated that referenced this pull request Jun 5, 2023
## Description

This PR removes `ReanimatedJSIModulePackage` class which was used for initializing Reanimated 2.4.1 and below.

Since 2.5.0, Reanimated automatically installs itself on Android.

You can remove `getJSIModulePackage()` method in `MainApplication.java` of your app.
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.

2 participants