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.4 #26774

Merged
merged 16 commits into from Feb 6, 2024

Conversation

gabrieldonadel
Copy link
Member

@gabrieldonadel gabrieldonadel commented Jan 29, 2024

Why

Follow up #26308 for react-native 0.73 upgrade

How

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

Test Plan

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

Checklist

@gabrieldonadel gabrieldonadel force-pushed the @gabrieldonadel/upgrade-react-native-0-73-3 branch from 10a2d22 to 24bb584 Compare January 30, 2024 16:19
@gabrieldonadel
Copy link
Member Author

I'm holding on merging this for now given this new version is throwing a warning as soon as the app starts and will probably cause some confusion for people.
A PR fixing this issue has already been opened on core facebook/react-native#42734 and it will be cherry-picked in 0.73.4

@hasanaktas
Copy link

Expo 50.0.5 version has been released, but react native version is still 0.73.2.

Do you have any information about when it will be published?

@brentvatne
Copy link
Member

brentvatne commented Feb 1, 2024

@hasanaktas - we intentionally did not land this yet because we don't believe it's good to upgrade to 0.73.3. as @gabrieldonadel pointed out, 0.73.3 includes a regression that results in a warning on startup on every ios app. it will be fixed by facebook/react-native#42734 and then when that is published 0.73.4, we will update the recommended version. this is a work in progress.

@joeyhotz
Copy link

joeyhotz commented Feb 6, 2024

0.73.4 out now fyi :)

@gabrieldonadel gabrieldonadel changed the title upgrade react-native 0.73.3 upgrade react-native 0.73.4 Feb 6, 2024
@gabrieldonadel gabrieldonadel force-pushed the @gabrieldonadel/upgrade-react-native-0-73-3 branch from 5ae7b9e to 73ad584 Compare February 6, 2024 16:48
@gabrieldonadel gabrieldonadel force-pushed the @gabrieldonadel/upgrade-react-native-0-73-3 branch from 73ad584 to 1ad265b Compare February 6, 2024 17:27
@gabrieldonadel gabrieldonadel merged commit 067e7b8 into sdk-50 Feb 6, 2024
26 checks passed
@gabrieldonadel gabrieldonadel deleted the @gabrieldonadel/upgrade-react-native-0-73-3 branch February 6, 2024 18:45
gabrieldonadel added a commit that referenced this pull request Feb 6, 2024
Follow up #26308 for react-native 0.73
upgrade

- update package versions
  - `react-native 0.73.2 -> 0.73.4`

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

<!--
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).
gabrieldonadel added a commit that referenced this pull request Feb 6, 2024
Follow up #26308 for react-native 0.73
upgrade

- update package versions
  - `react-native 0.73.2 -> 0.73.4`

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

<!--
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).
gabrieldonadel added a commit that referenced this pull request Feb 8, 2024
Follow up #26308 for react-native 0.73
upgrade

- update package versions
  - `react-native 0.73.2 -> 0.73.4`

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

<!--
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).
gabrieldonadel added a commit that referenced this pull request Feb 8, 2024
Follow up #26308 for react-native 0.73
upgrade

- update package versions
  - `react-native 0.73.2 -> 0.73.4`

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

<!--
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).
gabrieldonadel added a commit that referenced this pull request Feb 8, 2024
# Why

Backport #26774 to main

# How

`git cherry-pick 067e7b8 `

# 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).
@brentvatne brentvatne added the published Changes from the PR have been published to npm label Feb 16, 2024
@gabrieldonadel gabrieldonadel mentioned this pull request Mar 7, 2024
3 tasks
gabrieldonadel added a commit that referenced this pull request Mar 8, 2024
# 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

<!--
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

8 participants