ci: lint for ktlint compliance

Adds ktlint to pre-commit to include in the linting setup for all
projects. Additionally, the existing ktlint Gradle setup is simplified
and applied to all subprojects.

Author: languitar

.pre-commit-config.yaml

@@ -23,3 +23,11 @@ repos:
         stages: [commit-msg]
         additional_dependencies: ["@commitlint/config-angular"]
         args: ["--config", "./commitlint.config.js"]
+  - repo: https://github.com/macisamuele/language-formatters-pre-commit-hooks
+    rev: v2.10.0
+    hooks:
+      - id: pretty-format-kotlin
+        args:
+          # IMPORTANT: keep in sync with the version configured in build.gradle.kts
+          - --ktlint-version=0.50.0
+          - --autofix

build.gradle.kts

@@ -52,6 +52,8 @@ dependencies {
 subprojects {
     apply(plugin = "maven-publish")
     apply(plugin = "org.jetbrains.dokka")
+    apply(plugin = "org.jlleitschuh.gradle.ktlint")
+
     version = rootProject.version
     group = rootProject.group
 
@@ -61,6 +63,11 @@ subprojects {
         }
     }
 
+    configure<org.jlleitschuh.gradle.ktlint.KtlintExtension> {
+        // IMPORTANT: keep in sync with the version in .pre-commit-config.yaml
+        version.set("0.50.0")
+    }
+
     tasks.withType<org.jetbrains.kotlin.gradle.tasks.KotlinCompile>().all {
         kotlinOptions {
             jvmTarget = "11"

model-api/build.gradle.kts

@@ -1,31 +1,19 @@
-
 plugins {
     id("maven-publish")
     id("org.jetbrains.kotlin.multiplatform")
-    id("org.jlleitschuh.gradle.ktlint")
     kotlin("plugin.serialization")
 }
 
-configurations {
-    ktlint
-}
-
 description = "API to access models stored in Modelix"
 
 ktlint {
-    disabledRules.add("no-wildcard-imports")
-    outputToConsole.set(true)
-    this.filter {
-        this.exclude {
+    filter {
+        exclude {
             it.file.toPath().toAbsolutePath().startsWith(project(":ts-model-api").buildDir.toPath().toAbsolutePath())
         }
     }
 }
 
-tasks.named("check") {
-    dependsOn("ktlintCheck")
-}
-
 kotlin {
     jvm()
     js(IR) {
@@ -80,7 +68,14 @@ kotlin {
     }
 }
 
-listOf("sourcesJar", "runKtlintCheckOverJsMainSourceSet", "jsSourcesJar", "jsPackageJson", "compileKotlinJs", "jsProcessResources").forEach {
+listOf(
+    "sourcesJar",
+    "runKtlintCheckOverJsMainSourceSet",
+    "jsSourcesJar",
+    "jsPackageJson",
+    "compileKotlinJs",
+    "jsProcessResources",
+).forEach {
     tasks.named(it) {
         dependsOn(":ts-model-api:npm_run_build")
         dependsOn(":ts-model-api:patchKotlinExternals")

model-client/build.gradle.kts

@@ -2,29 +2,16 @@ plugins {
     `maven-publish`
     id("org.jetbrains.kotlin.multiplatform")
     id("com.diffplug.spotless")
-    id("org.jlleitschuh.gradle.ktlint")
     `java-library`
     jacoco
 }
 
-configurations {
-    ktlint
-}
-
 java {
     toolchain {
         languageVersion.set(JavaLanguageVersion.of(11))
     }
 }
 
-ktlint {
-    disabledRules.add("no-wildcard-imports")
-}
-
-tasks.named("check") {
-    dependsOn("ktlintCheck")
-}
-
 val kotlinCoroutinesVersion: String by rootProject
 val kotlinLoggingVersion: String by rootProject
 val ktorVersion: String by rootProject

modelql-client/build.gradle.kts

@@ -14,7 +14,6 @@
 plugins {
     kotlin("multiplatform")
     `maven-publish`
-    alias(libs.plugins.ktlint)
 }
 
 kotlin {

modelql-core/build.gradle.kts

@@ -15,7 +15,6 @@ plugins {
     kotlin("multiplatform")
     kotlin("plugin.serialization")
     `maven-publish`
-    alias(libs.plugins.ktlint)
 }
 
 kotlin {

modelql-html/build.gradle.kts

@@ -15,7 +15,6 @@ plugins {
     kotlin("multiplatform")
     kotlin("plugin.serialization")
     `maven-publish`
-    alias(libs.plugins.ktlint)
 }
 
 kotlin {

modelql-server/build.gradle.kts

@@ -1,7 +1,6 @@
 plugins {
     kotlin("jvm")
     `maven-publish`
-    alias(libs.plugins.ktlint)
 }
 
 dependencies {

modelql-typed/build.gradle.kts

@@ -15,7 +15,6 @@ plugins {
     kotlin("multiplatform")
     kotlin("plugin.serialization")
     `maven-publish`
-    alias(libs.plugins.ktlint)
 }
 
 kotlin {

modelql-untyped/build.gradle.kts

@@ -15,7 +15,6 @@ plugins {
     kotlin("multiplatform")
     kotlin("plugin.serialization")
     `maven-publish`
-    alias(libs.plugins.ktlint)
 }
 
 kotlin {