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

Update to Android Gradle plugin 7.0.2 and buildToolsVersion 3.0.3 #363

Merged
merged 1 commit into from
Oct 1, 2021

Conversation

vegaro
Copy link
Contributor

@vegaro vegaro commented Sep 16, 2021

Updated to the latest stable version of the Android Gradle plugin 7.0.2 and the build tools version to 3.0.3

Changes introduced by AGP 7 are outlined here https://android-developers.googleblog.com/2020/12/announcing-android-gradle-plugin.html

One of the main changes is that now we require JDK 11. In order to install it, the best way is to use sdkman. Instructions on that are here https://github.com/RevenueCat/purchases-android/blob/e2e4df2db2612d015a41db710e2a2afccd396c19/CONTRIBUTING.md#environment-setup

@vegaro vegaro marked this pull request as ready for review September 16, 2021 21:52
@vegaro vegaro requested a review from a team September 16, 2021 21:52
@@ -18,7 +18,6 @@ POM_DEVELOPER_NAME=RevenueCat, Inc.

android.useAndroidX=true
android.enableJetifier=true
android.enableR8.libraries = false
Copy link
Contributor

Choose a reason for hiding this comment

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

are we sure we no longer need this? do you remember why we had it in the first place?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So I looked into it, and found where I added it https://github.com/RevenueCat/purchases-android/pull/72/files

I think the reason I added it is to not use R8 and use ProGuard since people were still using ProGuard. That way we would get errors when trying to build release. Enabling/Disabling R8 has been deprecated since, so I think we can remove this line. It would actually not compile with this line on.

Screen Shot 2021-09-27 at 8 16 36 PM

So I think if by default AS will use R8 now, we should just be using R8.

Copy link
Contributor

@beylmk beylmk 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, just the one R8 question

@vegaro
Copy link
Contributor Author

vegaro commented Oct 1, 2021

@beylmk what do you think? :)

@beylmk
Copy link
Contributor

beylmk commented Oct 1, 2021

@beylmk what do you think? :)

Shoot, sorry i missed this! thanks for following up, i agree :) 🚢 it

@vegaro
Copy link
Contributor Author

vegaro commented Oct 1, 2021

yayyy updateeesss

@vegaro vegaro merged commit 054d7e3 into main Oct 1, 2021
@vegaro vegaro deleted the increase-versions branch October 1, 2021 01:18
@beylmk beylmk mentioned this pull request Oct 8, 2021
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.

2 participants