Skip to content
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

Merged
merged 7 commits into from Mar 8, 2024

Conversation

gabrieldonadel
Copy link
Member

Why

Follow up #26774 for react-native 0.73 upgrade

How

  • update package versions
    • react-native 0.73.4 -> 0.73.5

Test Plan

  • bare-expo ios / android
  • Expo Go
  • fabric ios / android
  • ci passed

Checklist

@expo-bot expo-bot added the bot: suggestions ExpoBot has some suggestions label Mar 7, 2024
@expo-bot expo-bot added bot: passed checks ExpoBot has nothing to complain about and removed bot: suggestions ExpoBot has some suggestions labels Mar 7, 2024
@gabrieldonadel gabrieldonadel force-pushed the @gabrieldonadel/upgrade-react-native-0-73-5 branch from 7e1bdb6 to 29fc9c7 Compare March 7, 2024 16:31
@gabrieldonadel gabrieldonadel marked this pull request as ready for review March 7, 2024 16:41
@gabrieldonadel gabrieldonadel force-pushed the @gabrieldonadel/upgrade-react-native-0-73-5 branch from 29fc9c7 to 92e201f Compare March 7, 2024 18:37
+ "expo-status-bar": "~1.11.1",
+ "react": "18.2.0",
+ "react-native": "0.73.2"
+ "react-native": "0.73.4"
Copy link
Member

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?

Copy link
Member Author

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

Copy link
Member Author

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

Copy link
Member Author

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"
Copy link
Member

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?

Copy link
Member

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)

@gabrieldonadel gabrieldonadel merged commit 57a8e74 into sdk-50 Mar 8, 2024
25 of 26 checks passed
@gabrieldonadel gabrieldonadel deleted the @gabrieldonadel/upgrade-react-native-0-73-5 branch March 8, 2024 13:58
gabrieldonadel added a commit that referenced this pull request Mar 8, 2024
# 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).
@brentvatne brentvatne added the published Changes from the PR have been published to npm label Mar 13, 2024
gabrieldonadel added a commit that referenced this pull request Mar 15, 2024
# 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).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: fingerprint changed bot: passed checks ExpoBot has nothing to complain about published Changes from the PR have been published to npm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants