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.
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
Update App to React Native 0.72 version #18507
Update App to React Native 0.72 version #18507
Changes from 30 commits
fb839b2
31a5993
aadce15
1853e3a
e317397
dc090e4
b7951d6
cc82588
00eb45d
94fffc7
4f6775d
0fb2aa0
675801f
b45c2f6
61c395b
ff1c818
4bbf36e
8c41f3c
ef3a33b
7c09b4e
20092ea
84196ee
6026a60
239def1
622e9f7
f18b635
3248140
e1cda52
50245aa
e7c0c61
5436537
0c3acdf
33bdacd
25696cd
4d42218
6a9fc44
5005579
dae067f
3e75ba2
3829399
ea2ffb2
952cad4
7b2292e
7b8b26f
058809d
f10fd80
12f60b4
2dc0e8c
043f3b7
8cc097b
4881cd4
7d40e26
2b07cf5
6c8e524
72a3520
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
This file was deleted.
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 think it will be better to set it as 3.x
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 agree with @parasharrajat that we should get on Ruby 3 if possible. However, I suggest that we lock to a specific version of Ruby and cocoapods. The
Podfile.lock
may have a diff if you dopod install
from a different version of cocoapods. I think that also broke pod caching in our CI, so we should lock that down to a specific version.Also, why remove the
.ruby-version
file?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.
@roryabraham @parasharrajat It was removed in the last version and now the ruby version is specified in
Gemfile
file, according to the upgrade helper tool.These two commits also give us some insight about what those decisions:
Yes, I agree as well (as long Ruby 3 works, needs testing). In fact, a common problem that we have here is that some contributors do
pod install
normally and some usebundle
to execute this task, which sometimes result in diffs inPodfile.lock
.Coming from this issue, I agree we should change the instructions to tell contributors to use only
bundle
to avoid these problems.