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

fix: update gradle files for 0.73 support #917

Merged
merged 2 commits into from
Feb 12, 2024
Merged

fix: update gradle files for 0.73 support #917

merged 2 commits into from
Feb 12, 2024

Conversation

oscb
Copy link
Contributor

@oscb oscb commented Feb 9, 2024

Current lib should just work with 0.73 but taking preliminary steps to keep up to date based on the work specified here
And in particular on the revised outputs of create-react-native-library CLI

  • Updated build.gradle and gradle.properties to match new CLI outputs
  • Added new AndroidManifest.xml without namespace
  • Added namespace in build.gradle only if Gradle 7.3+

Fixes #915

Current lib should just work with 0.73 but taking preliminary steps to keep up to date based on the work specified [here](react-native-community/discussions-and-proposals#671)
And in particular on the revised outputs of [create-react-native-library CLI](https://github.com/callstack/react-native-builder-bob/blob/fc8744092501ba00fd11bdb69e1ecb46fbb63c0f/packages/create-react-native-library/templates/native-common/android/build.gradle)

- Updated build.gradle and gradle.properties to match new CLI outputs
- Added new AndroidManifest.xml without namespace
- Added namespace in build.gradle only if Gradle 7.3+
Copy link

@chrisradek chrisradek left a comment

Choose a reason for hiding this comment

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

Looks good to me - did you already verify this doesn't break things for react-native < 0.73?


repositories {
google()
jcenter()
mavenCentral()

Choose a reason for hiding this comment

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

Was curious about this - found https://developer.android.com/build/jcenter-migration which explains why you needed to do this 😄
(And I guess it also just matches the new template)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was actually surprised that we hadn't done this before!
I remember it used to be very noisy in analytics-android/kotlin

@oscb
Copy link
Contributor Author

oscb commented Feb 9, 2024

@chrisradek

Looks good to me - did you already verify this doesn't break things for react-native < 0.73?

Yes, the E2E actually tests against 0.72
I cannot build personally anything less than 0.72 due to Xcode MacOS incompatibility. Apple is also removing the capability to submit apps not built with Xcode 15 in a few months so people will be upgrading to 0.72 at least.

Also of note that we're not bumping any major versions or using any particular features of Java/gradle so people should just be able to override with lower numbers if needed just as before

@oscb oscb merged commit 98d5b05 into master Feb 12, 2024
6 checks passed
@oscb oscb deleted the oscb/rn73 branch February 12, 2024 21:53
@oscb
Copy link
Contributor Author

oscb commented Feb 26, 2024

🎉 This PR is included in version 1.1.1 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

@oscb oscb added the released label Feb 26, 2024
@oscb
Copy link
Contributor Author

oscb commented Feb 26, 2024

🎉 This PR is included in version 2.19.0 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

@oscb
Copy link
Contributor Author

oscb commented Feb 26, 2024

🎉 This PR is included in version 1.3.1 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

@oscb
Copy link
Contributor Author

oscb commented Feb 26, 2024

🎉 This PR is included in version 1.1.1 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

@oscb
Copy link
Contributor Author

oscb commented Feb 26, 2024

🎉 This PR is included in version 0.6.1 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

@oscb
Copy link
Contributor Author

oscb commented Feb 26, 2024

🎉 This PR is included in version 1.1.1 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

@oscb
Copy link
Contributor Author

oscb commented Feb 26, 2024

🎉 This PR is included in version 0.4.1 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

@oscb
Copy link
Contributor Author

oscb commented Feb 26, 2024

🎉 This PR is included in version 0.7.1 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

@oscb
Copy link
Contributor Author

oscb commented Feb 26, 2024

🎉 This PR is included in version 1.2.0 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

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.

Please add support for React Native 0.73 and Expo SDK 50
2 participants