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 react-native 0.73.5 #27502
upgrade react-native 0.73.5 #27502
Conversation
7e1bdb6
to
29fc9c7
Compare
29fc9c7
to
92e201f
Compare
+ "expo-status-bar": "~1.11.1", | ||
+ "react": "18.2.0", | ||
+ "react-native": "0.73.2" | ||
+ "react-native": "0.73.4" |
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 this be 0.73.5?
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.
Yeah, not sure why et generate-bare-diffs
generated these changes
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.
So..., the et generate-bare-diffs
command uses origin to generate the diffs.
const diffCommand = `origin/${sdkToBranch(sdkFrom)}..origin/${sdkToBranch(sdkTo)}`; |
I'll merge this PR and then open a follow-up updating the template diffs again
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.
Here you go #27520
@@ -2476,7 +2476,7 @@ index 69e7eb9619..150bb72764 100644 | |||
+ "expo": "~50.0.0-preview.4", | |||
+ "expo-status-bar": "~1.11.0", | |||
+ "react": "18.2.0", | |||
+ "react-native": "0.73.2" | |||
+ "react-native": "0.73.4" |
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.
ditto re: shouldn't this be 0.73.5?
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.
(similar things in other diffs)
# Why Follow up of #27502 # How `et generate-bare-diffs` # Test Plan CI Should pass # Checklist <!-- Please check the appropriate items below if they apply to your diff. This is required for changes to Expo modules. --> - [ ] Documentation is up to date to reflect these changes (eg: https://docs.expo.dev and README.md). - [ ] Conforms with the [Documentation Writing Style Guide](https://github.com/expo/expo/blob/main/guides/Expo%20Documentation%20Writing%20Style%20Guide.md) - [ ] This diff will work correctly for `npx expo prebuild` & EAS Build (eg: updated a module plugin).
# Why Follow up #27502 for react-native 0.73 upgrade Template Bare Minimum Diffs will be addressed in a follow-up PR because of #27502 (comment) # How - update package versions - `react-native 0.73.5 -> 0.73.6` # Test Plan - bare-expo ios / android - Expo Go - fabric ios / android - ci passed # Checklist <!-- Please check the appropriate items below if they apply to your diff. This is required for changes to Expo modules. --> - [ ] Documentation is up to date to reflect these changes (eg: https://docs.expo.dev and README.md). - [ ] Conforms with the [Documentation Writing Style Guide](https://github.com/expo/expo/blob/main/guides/Expo%20Documentation%20Writing%20Style%20Guide.md) - [ ] This diff will work correctly for `npx expo prebuild` & EAS Build (eg: updated a module plugin).
Why
Follow up #26774 for react-native 0.73 upgrade
How
react-native 0.73.4 -> 0.73.5
Test Plan
Checklist
npx expo prebuild
& EAS Build (eg: updated a module plugin).