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

chore(android): gradle 8 #14014

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

AbdullahFaqeir
Copy link

chore(android): let there be gradle 8
chore(android): fixed deprecations in kroll-apt project
chore(android): made kroll-apt incremental to enhance build time

GitHub issue: #13579

This PR is dedicated to my mentor Nafe Abboushi.

chore(android): fixed deprecations in kroll-apt project
chore(android): made kroll-apt incremental to enhance build time
@m1ga m1ga self-requested a review March 23, 2024 20:25
Copy link
Contributor

@m1ga m1ga left a comment

Choose a reason for hiding this comment

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

Are the changes to the JAVA files necessary for the update? The more stuff that is changed, the more we have to test 😉

Also I wouldn't update the min and targetSDK with this PR. If we still can use 21 we should do that too. Same with target 34. I have a PR for that here #13940 as the broadcast receiver has to be adjusted.

So if that is not needed for the gradle PR please keep it as it is and do it in a separate PR (less changes, less testing).

I couldn't build a SDK locally. It's almost working but the last step titaniumPublication(MavenPublication) tries to publish it online instead of making a local repo? I've canceled it as it was uploading stuff 😄 Didn't want to publish something I don't need.

There aren't all cmake version available during build process. I have a different PR about that #13966 and I had to use 3.22.1 as that is available on all platforms.

fix(android): gradle 8 compatibility update to template build.gradle
fix(android): revert dependency updates
fix(android): revert minSdk and targetSdk
Copy link
Author

@AbdullahFaqeir AbdullahFaqeir left a comment

Choose a reason for hiding this comment

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

All reviews applied.

fix(android): update _build.js & _buildModule.js for new gradle
@m1ga m1ga mentioned this pull request Mar 26, 2024
@m1ga m1ga changed the title chore(android): let there be gradle 8 chore(android): gradle 8 Mar 26, 2024
@m1ga
Copy link
Contributor

m1ga commented Mar 26, 2024

I have to do some testing but the first round was very good and I was able to build kitchensink an my app now 👍

Review will follow once I have more time to test

@m1ga m1ga self-assigned this Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants