Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[packages] fix react-native 0.73 compatibility [1/3] #24018

Merged
merged 6 commits into from
Aug 19, 2023
Merged

Conversation

Kudo
Copy link
Contributor

@Kudo Kudo commented Aug 18, 2023

Why

fix react-native 0.73 compatibility. these are for AGP 8 support - react-native-community/discussions-and-proposals#671 (comment)

How

  • [packages][bare-expo] set jvm version only if AGP < 8
  • [av][dev-launcher] fix AGP 8 non transitive R issue
  • [dev-client][dev-launcher][dev-menu][core][notifications] fix AGP 8 non default buildConfig issue

Test Plan

ci passed in #23961 (https://github.com/expo/expo/pull/23961/checks)

Checklist

@expo-bot expo-bot added the bot: suggestions ExpoBot has some suggestions label Aug 18, 2023
@expo-bot expo-bot added bot: passed checks ExpoBot has nothing to complain about and removed bot: suggestions ExpoBot has some suggestions labels Aug 18, 2023
@Kudo Kudo changed the title [packages] fix react-native 0.73 compatibility [1/n] [packages] fix react-native 0.73 compatibility [1/3] Aug 18, 2023
@Kudo Kudo marked this pull request as ready for review August 18, 2023 14:38
Copy link
Contributor

@lukmccall lukmccall left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥🔥🔥

@Kudo Kudo merged commit 7ec55df into main Aug 19, 2023
@Kudo Kudo deleted the @kudo/fix-nightlies-1 branch August 19, 2023 07:45
EvanBacon pushed a commit that referenced this pull request Aug 23, 2023
# Why

fix react-native 0.73 compatibility. these are for AGP 8 support - react-native-community/discussions-and-proposals#671 (comment)

# How

- [packages][bare-expo] set jvm version only if AGP < 8
- [av][dev-launcher] fix AGP 8 non transitive R issue
- [dev-client][dev-launcher][dev-menu][core][notifications] fix AGP 8 non default buildConfig issue

# Test Plan

ci passed in #23961 (https://github.com/expo/expo/pull/23961/checks)
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bot: passed checks ExpoBot has nothing to complain about
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants