-
Notifications
You must be signed in to change notification settings - Fork 169
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 tools v30.3.0 #988
Conversation
.buckconfig
Outdated
@@ -25,6 +25,10 @@ | |||
[java] | |||
jar_spool_mode = direct_to_jar | |||
duplicates_log_level = fine | |||
compile_against_abis = false |
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.
Why?
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 plugin works in monorepo but okbuck tests are failing due to ASM in Buck trying to perform the class ABI on a java 17 class from a dependency. i'm working on a better fix
build.gradle
Outdated
@@ -1,7 +1,14 @@ | |||
buildscript { | |||
apply from: rootProject.file("dependencies.gradle") | |||
repositories { | |||
google() | |||
maven { | |||
url 'https://dl.google.com/android/maven2/' |
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.
doesn't google() exist?
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.
due to rocker and other breaking upgrades since we are on AGP 4.x, we need to set the property of the maven url to use pom instead of gradle metadata & ---we can't do this w/ google--- n/m, actually we can, so updated
b45630e
to
a911929
Compare
Update the Android tools version to allow for using Lint v31.3.1 or higher which is compatible with K2. Repos can choose their lint version, but to use a newer version, they need to use a compatible manifest merger version which had some breaking changes in v30.3.0.
A bump to Kotlin v1.5 was needed due to compatibility with transitive dependencies. Along the way, the sample project needed to be updates such as removing deprecated kotlin-extensions, disabling unused lints, & backporting removed methods.