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

chore(android)!: forward-port to new architecture method #1150

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mikehardy
Copy link
Collaborator

@mikehardy mikehardy commented Nov 25, 2024

Forward-port away from deprecated react-native method

The one we need to use now were added in react-native 0.65

facebook/react-native@dfa8eb0

react-native 0.65 was released in August 2021 so this shouldn't actually affect anyone but every time I think that I am surprised

Blocking this one pending other breaking changes, as there is not a pressing need to release it now, I just noticed it while troubleshooting and testing #1149

@mikehardy mikehardy added pending-merge Waiting on CI or question responses to merge, but otherwise ready Keep Open this label avoids the stale bot labels Nov 25, 2024
Copy link

codecov bot commented Nov 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.08%. Comparing base (ae2953b) to head (9ad5b86).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1150   +/-   ##
=======================================
  Coverage   77.08%   77.08%           
=======================================
  Files          32       32           
  Lines        1727     1727           
  Branches      556      579   +23     
=======================================
  Hits         1331     1331           
  Misses        395      395           
  Partials        1        1           

The method to forward-port to was added in react-native 0.65 so this
is breaking but should not actually affect anyone

BREAKING CHANGE: minimum required react-native now 0.65
@mikehardy mikehardy force-pushed the react-native-deprecation-drop-pre-0.65 branch from 9ad5b86 to 41fd91f Compare December 17, 2024 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Keep Open this label avoids the stale bot pending-merge Waiting on CI or question responses to merge, but otherwise ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant