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
[ios][android] Update react-native-gesture-handler to 2.7.0 #19362
Conversation
f112535
to
bb1ae89
Compare
bb1ae89
to
d7fa30f
Compare
d7fa30f
to
1af1c34
Compare
cmake { | ||
cppFlags "-O2 -frtti -fexceptions -Wall -Wno-unused-variable -fstack-protector-all" | ||
arguments "-DAPP_BUILD_DIR=${appProject.buildDir}", | ||
"-DREACT_NATIVE_DIR=${REACT_NATIVE_DIR}", |
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.
oh no! gesture-handler has cpp code now, even though it's for fabric only 🤦♂️
that would increase the complexity in versioning the vendored module.
i'm just taking a note that i should aware of this 😅
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.
most changes look good to me. just as mentioned i should follow up with versioning later.
there're some nit comments for out-of-sync lock files. you could also update them after the pr landed.
thanks for migrating this module as new style vendoring 🔥
@@ -58,6 +58,7 @@ const config: VendoringTargetConfig = { | |||
source: 'https://github.com/software-mansion/react-native-gesture-handler.git', | |||
semverPrefix: '~', | |||
ios: {}, | |||
android: {}, |
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.
would you like to remove the vendoring code in legacy.ts ?
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.
I planned to do that, but there are more libs to delete from there, so I'll do that separately
1af1c34
to
b84558e
Compare
Why
Upgrading the gesture handler for SDK 47
Closes ENG-6529
How
react-native-gesture-handler
to the new vendoring mechanismlib/build.gradle
from the source code so our vendored autolinking doesn't detect this as a separate library. This build.gradle should and will get removed from the repo (it's not used and uses pretty old versions that don't compile)et uvm -m react-native-gesture-handler -c 2.7.0
Test Plan
NCL examples seem to work fine