-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Android] Remove @override annotation to support RN 0.53-0.55 on detox 9.0 #922
[Android] Remove @override annotation to support RN 0.53-0.55 on detox 9.0 #922
Conversation
…ced until react-native 0.56 and breaks previous react-native versions
Thanks, @JamesWatling , I saw the same issue |
Thanks for this PR! This is weird, I can't see the CI integration here. @yershalom , any idea what happened ? |
@rotemmiz nope. |
@rotemmiz @yershalom weird. I just pushed an empty commit and it still isn't triggered... Is there anyway for it to be manually triggerred? |
@yershalom I started getting notifications from GitHub named wix/Detox instead of wix/detox. Maybe this is somehow related? |
It looks like the project name changed? If I look back to last month it was wix/detox, and now it's wix/Detox. Maybe the Jenkins needs to be updated too |
@rotemmiz who changed it? And yes, this is related |
No idea |
Change it back? |
I guess, yes |
Mk |
@JamesWatling can you please push another dummy commit? |
Any chance we could get a tag for this change? 9.0.2? Thanks! |
Yes. Looking forward to the next version bump to 9.0.2 that includes this change. |
remove @OverRide annotation on onBridgeDestroyed as it is not introduced until react-native 0.56 and breaks previous react-native versions
See #919
The NotThreadSafeBridgeIdleDebugListener interface doesn't add the method
onBridgeDestroyed()
until RN 0.56 this causes builds prior than that to break with an error as follows:This simply removes the
@Override
annotation without changing the expected functionality