Adding KtLint

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.yml

@@ -38,7 +38,7 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-gradle-
       - name: Build
-        run: ./gradlew build --no-daemon --stacktrace
+        run: ./gradlew ktlintCheck build --no-daemon --stacktrace
 
 env:
   GRADLE_OPTS: -Dkotlin.incremental=false -Dorg.gradle.jvmargs="-Xmx4g -XX:+HeapDumpOnOutOfMemoryError -Dfile.encoding=UTF-8 -XX:MaxMetaspaceSize=512m"

bugsnag-ios-link/build.gradle.kts

@@ -51,7 +51,10 @@ java {
     }
 }
 
+@Suppress("ktlint:standard:property-naming")
 val GROUP: String by project
+
+@Suppress("ktlint:standard:property-naming")
 val VERSION_NAME: String by project
 
 group = GROUP
@@ -63,4 +66,4 @@ mavenPublishing {
         project.properties["RELEASE_SIGNING_ENABLED"]?.toString()?.equals("false", ignoreCase = true) != true
     if (releaseSigningEnabled) signAllPublications()
     pomFromGradleProperties()
-}
+}

bugsnag-ios-link/src/main/kotlin/co/touchlab/crashkios/BugsnagLinkPlugin.kt

@@ -28,7 +28,8 @@ internal val Project.kotlinExtension: KotlinMultiplatformExtension get() = exten
 @Suppress("unused")
 class BugsnagLinkPlugin : Plugin<Project> {
     override fun apply(project: Project): Unit = project.withKotlinMultiplatformPlugin {
-        val linkerArgs = "-U _OBJC_CLASS_\$_BugsnagHandledState " +
+        val linkerArgs =
+            "-U _OBJC_CLASS_\$_BugsnagHandledState " +
                 "-U _OBJC_CLASS_\$_Bugsnag " +
                 "-U _OBJC_CLASS_\$_BugsnagStackframe " +
                 "-U _OBJC_CLASS_\$_FIRStackFrame " +

bugsnag/build.gradle.kts

@@ -19,7 +19,10 @@ plugins {
     id("com.vanniktech.maven.publish")
 }
 
+@Suppress("ktlint:standard:property-naming")
 val GROUP: String by project
+
+@Suppress("ktlint:standard:property-naming")
 val VERSION_NAME: String by project
 
 group = GROUP
@@ -88,15 +91,20 @@ kotlin {
 
 android {
     namespace = "co.touchlab.crashkios.bugsnag"
-    compileSdk = libs.versions.compileSdk.get().toInt()
+    compileSdk =
+        libs.versions.compileSdk
+            .get()
+            .toInt()
     defaultConfig {
-        minSdk = libs.versions.minSdk.get().toInt()
+        minSdk =
+            libs.versions.minSdk
+                .get()
+                .toInt()
     }
     compileOptions {
         sourceCompatibility = JavaVersion.VERSION_1_8
         targetCompatibility = JavaVersion.VERSION_1_8
     }
-
 }
 
 tasks.withType<KotlinCompile> {

bugsnag/src/androidMain/kotlin/co/touchlab/crashkios/bugsnag/BugsnagCallsActual.kt

@@ -18,4 +18,4 @@ actual class BugsnagCallsActual : BugsnagCalls {
     override fun setCustomValue(section: String, key: String, value: Any) {
         Bugsnag.addMetadata(section, key, value)
     }
-}
+}

bugsnag/src/appleMain/kotlin/co/touchlab/crashkios/bugsnag/BugsnagCallsActual.kt

@@ -6,7 +6,6 @@ import kotlinx.cinterop.ExperimentalForeignApi
 
 @OptIn(ExperimentalForeignApi::class)
 actual class BugsnagCallsActual : BugsnagCalls {
-
     override fun logMessage(message: String) {
         Bugsnag.leaveBreadcrumbWithMessage(message)
     }
@@ -45,4 +44,4 @@ actual class BugsnagCallsActual : BugsnagCalls {
             true
         }
     }
-}
+}

bugsnag/src/appleMain/kotlin/co/touchlab/crashkios/bugsnag/BugsnagConfig.kt

@@ -8,7 +8,7 @@ import com.rickclephas.kmp.nsexceptionkt.core.wrapUnhandledExceptionHook
 import kotlinx.cinterop.ExperimentalForeignApi
 import platform.Foundation.NSException
 
-public fun startBugsnag(config: BugsnagConfiguration){
+public fun startBugsnag(config: BugsnagConfiguration) {
     configureBugsnag(config)
     Bugsnag.startWithConfiguration(config)
     setBugsnagUnhandledExceptionHook()
@@ -53,6 +53,7 @@ public fun setBugsnagUnhandledExceptionHook(): Unit = wrapUnhandledExceptionHook
 /**
  * Feature flag used to mark the Kotlin termination crash.
  */
+@Suppress("ktlint:standard:property-naming")
 private const val kotlinCrashedFeatureFlag = "crashkios.kotlin_crashed"
 
 /**

bugsnag/src/commonMain/kotlin/co/touchlab/crashkios/bugsnag/BugsnagCalls.kt

@@ -2,9 +2,12 @@ package co.touchlab.crashkios.bugsnag
 
 interface BugsnagCalls {
     fun logMessage(message: String)
+
     fun sendHandledException(throwable: Throwable)
+
     fun sendFatalException(throwable: Throwable)
+
     fun setCustomValue(section: String, key: String, value: Any)
 }
 
-expect class BugsnagCallsActual() : BugsnagCalls
+expect class BugsnagCallsActual() : BugsnagCalls

bugsnag/src/commonMain/kotlin/co/touchlab/crashkios/bugsnag/BugsnagKotlin.kt

@@ -26,24 +26,20 @@ object BugsnagKotlin {
  * Call in startup code in an actual app. Tests should generally skip this. In Kotlin/Native, not calling this
  * for tests avoids linker issues.
  */
-fun enableBugsnag(){
+fun enableBugsnag() {
     BugsnagKotlin.implementation = BugsnagCallsActual()
 }
 
 internal object EmptyCalls : BugsnagCalls {
     override fun logMessage(message: String) {
-
     }
 
     override fun sendHandledException(throwable: Throwable) {
-
     }
 
     override fun sendFatalException(throwable: Throwable) {
-
     }
 
     override fun setCustomValue(section: String, key: String, value: Any) {
-
     }
-}
+}