-
Notifications
You must be signed in to change notification settings - Fork 47
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
MBS-12689 Support multiple sdks versions in a build check #1327
Conversation
} | ||
|
||
@Test | ||
fun `the same multiple versions - success`() { |
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 new scenario
} | ||
|
||
@Test | ||
fun `the same platform revision with legacy extension - success`() { |
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.
backward compatibility test
public data class AndroidSdkVersion( | ||
val compileSdkVersion: Int, | ||
val revision: Int, | ||
) : Serializable |
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.
Public because it's used as task's input
version( | ||
compileSdkVersion = 30, | ||
revision = 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.
Changes in API
ea0a7e7
to
c038a4f
Compare
Add missed scenario with different compileSdk in different modules.
Besides a known issue with build cache (https://issuetracker.google.com/issues/117789774) it also could lead to a broken sync in IDE. See examples in the issue description.
We can't use
android.builder.sdkDownload
in AGP 7.0.x due to a broken configuration cache (https://issuetracker.google.com/issues/189730917)