-
Notifications
You must be signed in to change notification settings - Fork 365
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
Upgrade targetSdkVersion to 30 #758
Conversation
5bfd441
to
c2f6998
Compare
c2f6998
to
abc0c3a
Compare
hi @jeorgearomal , because there will be no support from Google. Given the low adoption and the ongoing maintenance costs for us, we will bump the minSdkVersion soon but not this time(keep as is). In the future, after we upgrade minSdkVersion, if you still need to support lower minSdkVersion, you then have the option to use this version or any version before we upgrade minSdkVersion. Hope that helps! |
e2d01f5
to
6719f8c
Compare
compileSdkVersion 27 | ||
buildToolsVersion '28.0.3' | ||
compileSdkVersion 30 | ||
buildToolsVersion '30.0.3' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the .0.3
happens to be same?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it happens to be the same, it has to be a version exists, otherwise it won't build
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
got it. just making sure .0.3
is the version we want here.
src/androidTest/java/com/mixpanel/android/mpmetrics/MixpanelBasicTest.java
Outdated
Show resolved
Hide resolved
src/androidTest/java/com/mixpanel/android/mpmetrics/OptOutTest.java
Outdated
Show resolved
Hide resolved
src/androidTest/java/com/mixpanel/android/mpmetrics/TestUtils.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm. just a few comments.
95c4c12
to
fe90b3a
Compare
86497a9
to
0770f28
Compare
0770f28
to
585810f
Compare
Upgrade targetSdkVersion to 30 to meet Google Play's target API level requirement by Nov 2021
#756