Skip to content

Commit 6344a4c

Browse files
committed
Remove redundant jvmTarget, targetCompatibility & sourceCompatibility specifications
1 parent 1fec70d commit 6344a4c

File tree

3 files changed

+25
-48
lines changed

3 files changed

+25
-48
lines changed

app/build.gradle.kts

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,10 @@ plugins {
88
alias(libs.plugins.wifiwidget.hilt)
99
}
1010

11+
kotlin {
12+
jvmToolchain(libs.versions.java.get().toInt())
13+
}
14+
1115
android {
1216
val packageName = "com.w2sv.wifiwidget"
1317

@@ -73,15 +77,6 @@ android {
7377
}
7478
}
7579

76-
kotlinOptions {
77-
jvmTarget = libs.versions.java.get()
78-
}
79-
80-
compileOptions {
81-
sourceCompatibility = JavaVersion.VERSION_17
82-
targetCompatibility = JavaVersion.VERSION_17
83-
}
84-
8580
hilt {
8681
enableAggregatingTask = true // Fixes warning
8782
}

plugins/build.gradle.kts

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,7 @@
1-
import org.jetbrains.kotlin.gradle.tasks.KotlinCompile
2-
31
plugins {
42
`kotlin-dsl`
53
}
64

7-
java {
8-
sourceCompatibility = JavaVersion.VERSION_17
9-
targetCompatibility = JavaVersion.VERSION_17
10-
}
11-
tasks.withType<KotlinCompile>().configureEach {
12-
kotlinOptions {
13-
jvmTarget = JavaVersion.VERSION_17.toString()
14-
}
15-
}
165
kotlin {
176
jvmToolchain {
187
(this as JavaToolchainSpec).languageVersion.set(JavaLanguageVersion.of(libs.versions.java.get()))
Lines changed: 21 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1,39 +1,32 @@
1+
12
import com.android.build.gradle.BaseExtension
2-
import org.gradle.api.JavaVersion
33
import org.gradle.api.Project
44
import org.gradle.jvm.toolchain.JavaLanguageVersion
55
import org.gradle.kotlin.dsl.configure
6-
import org.gradle.kotlin.dsl.withType
76
import org.jetbrains.kotlin.gradle.dsl.KotlinProjectExtension
8-
import org.jetbrains.kotlin.gradle.tasks.KotlinCompile
97

108
internal fun Project.baseConfig() {
11-
tasks.withType<KotlinCompile>().configureEach {
12-
kotlinOptions {
13-
jvmTarget = JavaVersion.VERSION_17.toString()
14-
}
15-
}
16-
extensions.configure<KotlinProjectExtension> {
17-
jvmToolchain {
18-
languageVersion.set(JavaLanguageVersion.of(libs.findVersionInt("java")))
9+
with(extensions) {
10+
configure<KotlinProjectExtension> {
11+
jvmToolchain {
12+
languageVersion.set(JavaLanguageVersion.of(libs.findVersionInt("java")))
13+
}
1914
}
20-
}
21-
extensions.configure<BaseExtension> {
22-
namespace = "com.w2sv." + path.removePrefix(":").replace(':', '.')
23-
compileOptions {
24-
sourceCompatibility = JavaVersion.VERSION_17
25-
targetCompatibility = JavaVersion.VERSION_17
26-
}
27-
defaultConfig {
28-
minSdk = libs.findVersionInt("minSdk")
29-
targetSdk = libs.findVersionInt("compileSdk")
30-
testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
31-
}
32-
compileSdkVersion(libs.findVersionInt("compileSdk"))
33-
testOptions {
34-
unitTests.isReturnDefaultValues = true
35-
animationsDisabled = true
36-
unitTests.isIncludeAndroidResources = true
15+
configure<BaseExtension> {
16+
// Set namespace to com.w2sv.<module-name>
17+
namespace = "com.w2sv." + path.removePrefix(":").replace(':', '.')
18+
19+
defaultConfig {
20+
minSdk = libs.findVersionInt("minSdk")
21+
targetSdk = libs.findVersionInt("compileSdk")
22+
testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
23+
}
24+
compileSdkVersion(libs.findVersionInt("compileSdk"))
25+
testOptions {
26+
unitTests.isReturnDefaultValues = true
27+
animationsDisabled = true
28+
unitTests.isIncludeAndroidResources = true
29+
}
3730
}
3831
}
3932
}

0 commit comments

Comments
 (0)