Updating KtLint (#440)

Author: KevinSchildhorn

.editorconfig

@@ -0,0 +1,23 @@
+# .editorconfig
+root = true
+
+[*.{kt,kts}]
+end_of_line = lf
+ij_kotlin_packages_to_use_import_on_demand = true
+ij_kotlin_allow_trailing_comma = true
+ij_kotlin_allow_trailing_comma_on_call_site = true
+ij_kotlin_imports_layout = *
+ij_kotlin_indent_before_arrow_on_new_line = false
+ij_kotlin_line_break_after_multiline_when_entry = true
+indent_size = 4
+indent_style = space
+insert_final_newline = true
+ktlint_argument_list_wrapping_ignore_when_parameter_count_greater_or_equal_than = 8
+ktlint_chain_method_rule_force_multiline_when_chain_operator_count_greater_or_equal_than = 4
+ktlint_code_style = android_studio
+ktlint_enum_entry_name_casing = upper_or_camel_cases
+ktlint_function_naming_ignore_when_annotated_with = Composable
+ktlint_function_signature_body_expression_wrapping = default
+ktlint_ignore_back_ticked_identifier = false
+max_line_length = 140
+parameter-list-wrapping = true

.github/workflows/build_mac.yml

@@ -33,7 +33,7 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-gradle-
       - name: Build
-        run: ./gradlew build --no-daemon --stacktrace --build-cache
+        run: ./gradlew ktlintCheck build --no-daemon --stacktrace --build-cache
         env:
           S3_BUILD_CACHE_AWS_REGION: ${{ secrets.S3_BUILD_CACHE_AWS_REGION }}
           S3_BUILD_CACHE_BUCKET_NAME: ${{ secrets.S3_BUILD_CACHE_BUCKET_NAME }}

.github/workflows/build_win.yml

@@ -39,7 +39,7 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-gradle-
       - name: Build
-        run: ./gradlew mingwX64Test --no-daemon --stacktrace --build-cache
+        run: ./gradlew ktlintCheck mingwX64Test --no-daemon --stacktrace --build-cache
         env:
           S3_BUILD_CACHE_AWS_REGION: ${{ secrets.S3_BUILD_CACHE_AWS_REGION }}
           S3_BUILD_CACHE_BUCKET_NAME: ${{ secrets.S3_BUILD_CACHE_BUCKET_NAME }}

.idea/codeStyles/Project.xml

@@ -23,8 +23,9 @@ plugins {
     alias(libs.plugins.dokka) apply false
     alias(libs.plugins.touchlab.docusaurus.template)
     alias(libs.plugins.android.library) apply false
-
+    id("org.jlleitschuh.gradle.ktlint") version "12.2.0" apply false
 }
+
 apiValidation {
     nonPublicMarkers.add("co.touchlab.kermit.ExperimentalKermitApi")
 //    ignoredProjects.addAll(listOf("kermit-gradle-plugin", "kermit-ir-plugin", "kermit-ir-plugin-native"))
@@ -46,10 +47,29 @@ allprojects {
 
 allprojects {
     apply(plugin = "org.jetbrains.dokka")
+
     repositories {
         mavenCentral()
         google()
     }
     tasks.getByName("dokkaHtml").dependsOn(":kermit:transformIosMainCInteropDependenciesMetadataForIde")
 }
 
+subprojects {
+    apply(plugin = "org.jlleitschuh.gradle.ktlint")
+
+    configure<org.jlleitschuh.gradle.ktlint.KtlintExtension> {
+        version.set("1.4.0")
+        enableExperimentalRules.set(true)
+        verbose.set(true)
+        filter {
+            exclude { it.file.path.contains("build/") }
+        }
+    }
+
+    afterEvaluate {
+        tasks.named("check") {
+            dependsOn(tasks.getByName("ktlintCheck"))
+        }
+    }
+}

build.gradle.kts

@@ -59,4 +59,4 @@ android {
 
 tasks.withType<KotlinCompile> {
     kotlinOptions.jvmTarget = "1.8"
-}
+}

extensions/kermit-bugsnag/build.gradle.kts

@@ -25,7 +25,7 @@ import co.touchlab.kermit.Tag
 class BugsnagLogWriter(
     private val minSeverity: Severity = Severity.Info,
     private val minCrashSeverity: Severity? = Severity.Warn,
-    private val messageStringFormatter: MessageStringFormatter = DefaultFormatter
+    private val messageStringFormatter: MessageStringFormatter = DefaultFormatter,
 ) : LogWriter() {
 
     private val bugsnagCalls: BugsnagCalls = BugsnagCallsActual()
@@ -42,7 +42,7 @@ class BugsnagLogWriter(
 
     override fun log(severity: Severity, message: String, tag: String, throwable: Throwable?) {
         bugsnagCalls.logMessage(
-            messageStringFormatter.formatMessage(severity, Tag(tag), Message(message))
+            messageStringFormatter.formatMessage(severity, Tag(tag), Message(message)),
         )
         if (throwable != null && minCrashSeverity != null && severity >= minCrashSeverity) {
             bugsnagCalls.sendHandledException(throwable)

extensions/kermit-bugsnag/src/commonMain/kotlin/co/touchlab/kermit/bugsnag/BugsnagLogWriter.kt

@@ -59,4 +59,4 @@ android {
 
 tasks.withType<KotlinCompile> {
     kotlinOptions.jvmTarget = "1.8"
-}
+}

extensions/kermit-crashlytics/build.gradle.kts

@@ -25,7 +25,7 @@ import co.touchlab.kermit.Tag
 class CrashlyticsLogWriter(
     private val minSeverity: Severity = Severity.Info,
     private val minCrashSeverity: Severity? = Severity.Warn,
-    private val messageStringFormatter: MessageStringFormatter = DefaultFormatter
+    private val messageStringFormatter: MessageStringFormatter = DefaultFormatter,
 ) : LogWriter() {
 
     private val crashlyticsCalls: CrashlyticsCalls = CrashlyticsCallsActual()
@@ -42,7 +42,7 @@ class CrashlyticsLogWriter(
 
     override fun log(severity: Severity, message: String, tag: String, throwable: Throwable?) {
         crashlyticsCalls.logMessage(
-            messageStringFormatter.formatMessage(severity, Tag(tag), Message(message))
+            messageStringFormatter.formatMessage(severity, Tag(tag), Message(message)),
         )
         if (throwable != null && minCrashSeverity != null && severity >= minCrashSeverity) {
             crashlyticsCalls.sendHandledException(throwable)

extensions/kermit-crashlytics/src/commonMain/kotlin/co/touchlab/kermit/crashlytics/CrashlyticsLogWriter.kt

@@ -73,4 +73,4 @@ android {
 
 tasks.withType<KotlinCompile> {
     kotlinOptions.jvmTarget = "1.8"
-}
+}