mirror of
https://github.com/apple/pkl.git
synced 2026-04-25 09:48:41 +02:00
Fix build performance issues of spotless tasks (#722)
Using more efficient glob expressions and removing `targetExclude()` greatly speeds up performance on some OSes.
This commit is contained in:
@@ -129,7 +129,6 @@ private val licenseHeaderFile by lazy {
|
|||||||
|
|
||||||
private fun KotlinGradleExtension.configureFormatter() {
|
private fun KotlinGradleExtension.configureFormatter() {
|
||||||
ktfmt(libs.versions.ktfmt.get()).googleStyle()
|
ktfmt(libs.versions.ktfmt.get()).googleStyle()
|
||||||
targetExclude("**/generated/**", "**/build/**")
|
|
||||||
licenseHeaderFile(licenseHeaderFile, "([a-zA-Z]|@file|//)")
|
licenseHeaderFile(licenseHeaderFile, "([a-zA-Z]|@file|//)")
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -140,15 +139,17 @@ spotless {
|
|||||||
if (project === rootProject) {
|
if (project === rootProject) {
|
||||||
kotlinGradle {
|
kotlinGradle {
|
||||||
configureFormatter()
|
configureFormatter()
|
||||||
target("*.kts", "buildSrc/**/*.kts")
|
target("*.kts", "buildSrc/*.kts", "buildSrc/src/*/kotlin/**/*.kts")
|
||||||
}
|
}
|
||||||
kotlin {
|
kotlin {
|
||||||
ktfmt(libs.versions.ktfmt.get()).googleStyle()
|
ktfmt(libs.versions.ktfmt.get()).googleStyle()
|
||||||
targetExclude("**/generated/**", "**/build/**")
|
target("buildSrc/src/*/kotlin/**/*.kt")
|
||||||
target("buildSrc/**/*.kt")
|
|
||||||
licenseHeaderFile(licenseHeaderFile)
|
licenseHeaderFile(licenseHeaderFile)
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
kotlinGradle { configureFormatter() }
|
kotlinGradle {
|
||||||
|
configureFormatter()
|
||||||
|
target("*.kts")
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -42,12 +42,12 @@ artifacts {
|
|||||||
spotless {
|
spotless {
|
||||||
java {
|
java {
|
||||||
googleJavaFormat(libs.versions.googleJavaFormat.get())
|
googleJavaFormat(libs.versions.googleJavaFormat.get())
|
||||||
targetExclude("**/generated/**", "**/build/**")
|
target("src/*/java/**/*.java")
|
||||||
licenseHeaderFile(rootProject.file("buildSrc/src/main/resources/license-header.star-block.txt"))
|
licenseHeaderFile(rootProject.file("buildSrc/src/main/resources/license-header.star-block.txt"))
|
||||||
}
|
}
|
||||||
kotlin {
|
kotlin {
|
||||||
ktfmt(libs.versions.ktfmt.get()).googleStyle()
|
ktfmt(libs.versions.ktfmt.get()).googleStyle()
|
||||||
targetExclude("**/generated/**", "**/build/**")
|
target("src/*/kotlin/**/*.kt")
|
||||||
licenseHeaderFile(rootProject.file("buildSrc/src/main/resources/license-header.star-block.txt"))
|
licenseHeaderFile(rootProject.file("buildSrc/src/main/resources/license-header.star-block.txt"))
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user