ci: Added pre-commit-hook

.githooks/linkHooks.bat

@@ -0,0 +1,4 @@
+@echo off
+echo Changing git config for hooks to .githooks
+call git config --local core.hooksPath .githooks/
+echo Complete!

.githooks/linkHooks.sh

@@ -0,0 +1,5 @@
+#!/bin/sh
+set echo of
+echo Changing git config for hooks to .githooks
+git config --local core.hooksPath .githooks/
+echo Complete!

.githooks/pre-commit

@@ -0,0 +1,17 @@
+#!/bin/sh
+
+echo "Running detekt check"
+OUTPUT="/tmp/detekt-$(date +%s)"
+./gradlew detekt > $OUTPUT
+EXIT_CODE=$?
+if [ $EXIT_CODE -ne 0 ]; then
+  cat $OUTPUT
+  rm $OUTPUT
+  echo
+  echo "***********************************************"
+  echo "                 Detekt failed                 "
+  echo " Please fix the above issues before committing "
+  echo "***********************************************"
+  exit $EXIT_CODE
+fi
+rm $OUTPUT

build.gradle.kts

@@ -21,15 +21,6 @@ repositories {
     mavenCentral()
 }
 
-subprojects {
-    afterEvaluate {
-        if (tasks.findByName("detekt") != null) {
-            tasks.withType<org.jetbrains.kotlin.gradle.tasks.KotlinCompile> {
-                dependsOn(tasks["detekt"])
-            }
-        }
-    }
-}
 
 tasks.named("dependencyUpdates", DependencyUpdatesTask::class.java).configure {
 

config/detekt.yml

@@ -189,7 +189,7 @@ formatting:
     autoCorrect: true
   ParameterListWrapping:
     excludes: ['**/test/**', '**/*.Test.kt', '**/*.Spec.kt']
-    active: false
+    active: true
     autoCorrect: true
     indentSize: 4
   SpacingAroundColon:

test_runner/src/main/kotlin/ftl/ios/Xctestrun.kt

@@ -92,8 +92,8 @@ object Xctestrun {
         }
 
     private fun findTestNamesForTarget(
-            testTarget: String,
-            xctestrun: File
+        testTarget: String,
+        xctestrun: File
     ): List<String> =
         testsForTarget(
             testDictionary = parse(xctestrun)[testTarget]