-
Notifications
You must be signed in to change notification settings - Fork 302
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
Upgrade to React Native 0.73.6 #6596
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 5 Ignored Deployments
|
5c490c4
to
2be1eba
Compare
2be1eba
to
51a469d
Compare
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
Desktop Bundle Checks
Mobile Bundle Checks🚀 main.ios.jsbundle bundle size decreased (59.6mb -> 58.7mb). Thanks ❤️ |
exclude group:'com.squareup.okhttp3', module:'okhttp' | ||
} | ||
debugImplementation("com.facebook.flipper:flipper-fresco-plugin:${FLIPPER_VERSION}") | ||
implementation("com.facebook.react:flipper-integration") |
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.
😍
@@ -105,6 +105,8 @@ describe.skip("testing xpub native segwit transactions", () => { | |||
.toString("hex"); | |||
|
|||
psbt.addInput({ | |||
// eslint-disable-next-line @typescript-eslint/ban-ts-comment | |||
//@ts-expect-error |
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.
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.
Thanks, I will update it
e65bf69
to
bbcc9bc
Compare
bbcc9bc
to
1cf0175
Compare
a26c1ec
to
9f45b73
Compare
27f68f8
to
60774a4
Compare
60774a4
to
5e69111
Compare
5e69111
to
623366c
Compare
* feat: Upgrade dependencies * fix: iOS build * fix: Android build and migrate to Kotlin * feat: use Metro Symlink resolver and cleanup our custom resolver * fix: Update Tooling and CI * chore: cleanup unused components and libs * fix: linter * fix: test on our mac runners * fix: types for react-native-webview * fix: revert react-native-webview version * fix: run ios cli * fix: increase memory for test libraries job
✅ Checklist
npx changeset
was attached.📝 Description
The goal of this PR is to update to React Native 73.x and remove custom resolvers we use to favor symlink support introduced in latest react native version
Most impacting libraries update:
And many other dependencies...
I also removed two UI components and related libraries that we don't use anymore:
I upgraded Metro to the latest version and fine-tuned it to allow working with symlinks on our monorepo. This allows us to remove metro-extra-config package and rely directly on Metro to resolve our dependencies.
I migrated Java files to Kotlin for Android
Requirements for local execution
brew upgrade
rm -rf ~/.cocoapods/
)bundle install
in ledger-live-mobile folderpnpm clean
pnpm store prune
❓ Context
🧐 Checklist for the PR Reviewers