-
Notifications
You must be signed in to change notification settings - Fork 576
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
RJS-2913: Supporting React Native v0.76.0 #6912
Conversation
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.
Not too familiar with the intricacies here but looks reasonable, just one note about the commented code.
@@ -23,7 +23,7 @@ import { act, fireEvent, render, renderHook, waitFor } from "@testing-library/re | |||
import { AppProvider, useApp } from "../AppProvider"; | |||
import { Realm } from "realm"; | |||
|
|||
jest.mock("react-native/Libraries/Animated/NativeAnimatedHelper"); | |||
// jest.mock("react-native/Libraries/Animated/NativeAnimatedHelper"); |
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.
Should it be left as a comment or just removed?
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.
To be honest, I don't understand the original need for this line and the file is missing in the latest version of React Native, leading to a test failure.
I left it as a comment to make it easier to search for and blame if it starts failing 🙈
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.
Fair enough, although maybe it'd be good to have a short comment just stating that
fbc99bd
to
2a6d2ae
Compare
2a6d2ae
to
e95732a
Compare
What, How & Why?
This upgrades our test app to use React Native 0.76.0 and closes #6910 and #6908.
☑️ ToDos
Compatibility
label is updated or copied from previous entryCOMPATIBILITY.md
package.json
s (if updating internal packages)Breaking
label has been applied or is not necessary@realm/devdocs
if documentation changes are needed