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

[2/n] upgrade react-native 0.73 #25488

Merged
merged 10 commits into from
Nov 22, 2023

Conversation

gabrieldonadel
Copy link
Member

@gabrieldonadel gabrieldonadel commented Nov 20, 2023

Why

Follow up #25453 for react-native 0.73 upgrade
close ENG-9739

How

  • update package versions
    • react-native 0.73.0-rc.4 -> 0.73.0-rc.5
  • Bump test-suite iOS build timeout to 45 min
  • Publish dev home
  • Remove @react-native/dev-middleware patch

Test Plan

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

Checklist

Copy link

linear bot commented Nov 20, 2023

@expo-bot expo-bot added the bot: suggestions ExpoBot has some suggestions label Nov 20, 2023
@gabrieldonadel gabrieldonadel force-pushed the @gabrieldonadel/upgrade-react-native-0-73-part-2 branch from 3dee83b to 388e06a Compare November 21, 2023 12:28
@gabrieldonadel gabrieldonadel force-pushed the @gabrieldonadel/upgrade-react-native-0-73-part-3 branch 2 times, most recently from 6d60e2c to 4d3544f Compare November 21, 2023 18:06
@gabrieldonadel gabrieldonadel changed the title [3/n] upgrade react-native 0.73 [2/n] upgrade react-native 0.73 Nov 21, 2023
@gabrieldonadel gabrieldonadel force-pushed the @gabrieldonadel/upgrade-react-native-0-73-part-3 branch from 4d3544f to 77cef72 Compare November 21, 2023 18:07
@gabrieldonadel
Copy link
Member Author

I'm inverting the order between this PR and #25453 to fix our CI

@gabrieldonadel gabrieldonadel changed the base branch from @gabrieldonadel/upgrade-react-native-0-73-part-2 to main November 21, 2023 18:08
@gabrieldonadel gabrieldonadel force-pushed the @gabrieldonadel/upgrade-react-native-0-73-part-3 branch from 639b6fc to 347f9e0 Compare November 21, 2023 18:45
@gabrieldonadel gabrieldonadel force-pushed the @gabrieldonadel/upgrade-react-native-0-73-part-3 branch from 347f9e0 to f4d5773 Compare November 21, 2023 18:56
@expo-bot
Copy link
Collaborator

Hi there! 👋 I'm a bot whose goal is to ensure your contributions meet our guidelines.

I've found some issues in your pull request that should be addressed (click on them for more details) 👇

⚠️ Suggestion: Missing changelog entries


Your changes should be noted in the changelog. Read Updating Changelogs guide and consider adding an appropriate entry to the following changelogs:


Generated by ExpoBot 🤖 against 61ccd47

@gabrieldonadel
Copy link
Member Author

Android Instrumentation Tests is failing due to unrelated changes, going to merge as is

@gabrieldonadel gabrieldonadel merged commit a01b919 into main Nov 22, 2023
22 of 23 checks passed
@gabrieldonadel gabrieldonadel deleted the @gabrieldonadel/upgrade-react-native-0-73-part-3 branch November 22, 2023 01:25
gabrieldonadel added a commit that referenced this pull request Nov 29, 2023
# Why

Follow up #25488 for react-native 0.73
upgrade
close ENG-9739

# How

- update package versions
  - `react-native 0.73.0-rc.5 -> 0.73.0-rc.6`  
- Publish dev home
- upgrade project templates based on
[upgrade-helper](https://react-native-community.github.io/upgrade-helper/?from=0.73.0-rc.5&to=0.73.0-rc.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)
- [X] This diff will work correctly for `npx expo prebuild` & EAS Build
(eg: updated a module plugin).
onizam95 pushed a commit to onizam95/expo-av-drm that referenced this pull request Jan 15, 2024
# Why

Follow up expo#25453 for react-native 0.73
upgrade
close ENG-9739

# How

- update package versions
  - `react-native 0.73.0-rc.4 -> 0.73.0-rc.5` 
- Bump test-suite iOS build timeout to 45 min 
- Publish dev home
- Remove @react-native/dev-middleware patch

# 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)
- [x] This diff will work correctly for `npx expo prebuild` & EAS Build
(eg: updated a module plugin).
onizam95 pushed a commit to onizam95/expo-av-drm that referenced this pull request Jan 15, 2024
# Why

Follow up expo#25488 for react-native 0.73
upgrade
close ENG-9739

# How

- update package versions
  - `react-native 0.73.0-rc.5 -> 0.73.0-rc.6`  
- Publish dev home
- upgrade project templates based on
[upgrade-helper](https://react-native-community.github.io/upgrade-helper/?from=0.73.0-rc.5&to=0.73.0-rc.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)
- [X] 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants