Skip to content

Conversation

@alexsavg
Copy link
Contributor

No description provided.

id 'org.jetbrains.kotlin.android'
}

apply plugin: 'kotlin-kapt'

Choose a reason for hiding this comment

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

How come we are using two different ways to apply plugins in this project (the plugins block and the apply plugin directive)?

Choose a reason for hiding this comment

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

How come we are using two different ways to apply plugins in this project (the plugins block and the apply plugin directive)?

compileOptions {
sourceCompatibility JavaVersion.VERSION_1_8
targetCompatibility JavaVersion.VERSION_1_8
sourceCompatibility JavaVersion.VERSION_17

Choose a reason for hiding this comment

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

This (and minsdk) seems like it will significantly restrict what phones this sample will run on. Since this is a sample we intend for other people, can we make it run more broadly?

implementation "androidx.appsearch:appsearch-local-storage:$appsearch_version"
implementation "androidx.appsearch:appsearch-platform-storage:$appsearch_version"
kapt "androidx.appsearch:appsearch-compiler:$appsearch_version"
//api 'androidx.annotation:annotation:1.6.0'

Choose a reason for hiding this comment

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

Not quite clear what this new comment is intended to communicate -- could you clarify?

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.

4 participants