-
Notifications
You must be signed in to change notification settings - Fork 3.7k
Upgrade React Native to v0.83 #79962
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
Merged
Merged
Changes from all commits
Commits
Show all changes
87 commits
Select commit
Hold shift + click to select a range
7c7511a
chore: migrate RN patches
war-in e609f08
Merge branch 'main' into war-in/rn-83
war-in f9204b2
chore: update more libraries
war-in dfcd6d7
wip
war-in 49310f3
fix: clean-up lottie patch
war-in 10568ff
fix: InteractionManager patch
war-in 023cb49
chore: bump rock
war-in 54fdf07
Merge branch 'main' into war-in/rn-83
war-in e167025
first steps into iOS
war-in bbb7ee9
Merge branch 'main' into war-in/rn-83
war-in f2350a4
Merge branch 'war-in/bump-reanimated-421' into war-in/rn-83
war-in 7d45777
Merge branch 'main' into war-in/rn-83
war-in 5a5566b
fix: android build
war-in 3fdbd5e
fix: ios build
war-in 9841778
chore: revert unnecessary rock bump
war-in 7cc263c
chore: rename patches
war-in 4fe581e
fix: remove unnecessary `react-native-nitro-modules` bump
war-in 62b3543
Merge branch 'main' into war-in/rn-83
war-in 1a2a4d9
chore: post-merge fixes
war-in 64a081a
fix: standalone builds
war-in c9c3615
Merge branch 'main' into war-in/rn-83
war-in 97f608d
chore: post-merge fixes
war-in 8cd82ac
fix: android build
war-in 8b850c4
Merge branch 'main' into war-in/rn-83
war-in 76c1f78
fix: make iOS work
war-in ca408a5
Merge branch 'refs/heads/main' into war-in/rn-83
war-in c8d2797
chore: remove outdated patch
war-in 91262cb
fix: iOS crash when switching experiences
war-in 9d07843
Merge branch 'refs/heads/main' into war-in/rn-83
war-in 107729d
fix: add patches to details
war-in c22f696
chore: update podfile
war-in c2dadc8
fix: typechesk
war-in 2b86344
Merge branch 'main' into war-in/rn-83
war-in cba8917
fix: typecheck
war-in b60293f
fix: tests
war-in 5c25f44
fix: patches
war-in 11fac92
fix: remove outdated patch
war-in ac0f5ef
chore: update podfile.lock
war-in 125e351
fix: jest setup
war-in 756fb4d
fix: revert unrelated changes
war-in 29c3a64
fix: prettier
war-in 8be492c
fix: podfile
war-in 872c74f
fix: update `expo-image` to fix android release build error
war-in 7716f14
Merge branch 'main' into war-in/rn-83
war-in 2036850
chore: update patch
war-in 4d22ada
Merge branch 'main' into war-in/rn-83
war-in 55557e5
fix: android release build by updating firebase-crashlytics-gradle ve…
war-in 1cc6eff
Merge branch 'main' into war-in/rn-83
war-in c5896b4
Merge branch 'main' into war-in/rn-83
war-in cef0ee8
Merge branch 'main' into war-in/rn-83
war-in 5106918
Merge branch 'main' into war-in/rn-83
war-in f89c378
fix: fetching files on Android
war-in 6223687
chore: update expo to 55
war-in 22bc02d
chore: update Podfile.lock
war-in 8104eef
Merge branch 'main' into war-in/rn-83
war-in e7b1496
fix: navigation to concierge
war-in e0f5d75
Merge branch 'main' into war-in/rn-83
war-in df72486
Merge branch 'refactor/replace-SelectionListWithSections-with-new-Sel…
war-in b0011ad
fix: textInput autofocus when navigating back
war-in a9b2bb6
Merge branch 'refs/heads/main' into war-in/rn-83
war-in c2f1803
chore: rename patches
war-in 1973b6e
chore: update @expo/metro-runtime
war-in d35ec89
fix: prettier
war-in 81f3735
fix: rename patches
war-in c327886
Merge branch 'main' into war-in/rn-83
war-in a579a92
fix: lazy loading illustrations on Android dev
war-in fd2730f
fix: package-lock.json
war-in e4ce62f
chore: add RN patch details
war-in f36ce97
Merge branch 'main' into war-in/rn-83
war-in f775082
Merge branch 'main' into war-in/rn-83
war-in 6e4cc21
fix: wrong avatar image orientation
war-in 7bace85
merge
war-in 7908891
chore: post-merge fixed
war-in dd674aa
chore: fix patch
war-in 84cc794
chore: fix typecheck
war-in f6a5af7
chore: add upstream PR
war-in 4a51d12
fix: gh actions check
war-in 3e1e3fd
fix: lint
war-in ef42da2
Merge branch 'main' into war-in/rn-83
war-in dc6a4e1
chore: fix locks
war-in 73291f9
Merge branch 'main' into war-in/rn-83
war-in 681fa9e
chore: add upstream PRs
war-in 71d1ac7
Merge branch 'main' into war-in/rn-83
war-in b074b92
chore: post-merge fixes
war-in 68b4b1d
Merge branch 'refs/heads/main' into war-in/rn-83
war-in 6bc063f
Merge branch 'main' into war-in/rn-83
war-in 83089bb
Merge branch 'main' into war-in/rn-83
war-in File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
1,968 changes: 922 additions & 1,046 deletions
1,968
.github/actions/javascript/authorChecklist/index.js
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Binary file not shown.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,4 @@ | ||
| import groovy.xml.XmlSlurper | ||
| import groovy.json.JsonSlurper | ||
| import java.util.concurrent.TimeUnit | ||
|
|
||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Sounds like bumping to gradle 9.0.0 gives us some tasks to do:
applicationVariantsAPI is now deprecated (replaced byapplicationComponentsAPI). Looks like we're using this in ourbuild.gradle.kotlin-androidplugin is now built-in, and we can/should removeapply plugin: 'kotlin-android'2.1.20, so we should bump that at least to2.2AI list for next steps:
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.
Thank you for the review! I agree that we should put some love into our gradle setup but I don't think we should update AGP to v9
I'm usually using React Native Upgrade Helper when upgrading RN and it suggested upgrading gradle to v9. However, when I tried upgrading AGP through Android Studio I saw this information
So, I'd rather stay on gradle v9 until the helper recommends the update.
What we can do is run
./gradlew help --warning-mode=allon main and fix project-related warnings. But I think this can be done in a separate issueHow do you feel about it? Do you have any additional thoughts?
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 suppose if everything's working, that's fine