Skip to content

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
@codecov
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.

2 participants