Build optimizations (#163)

### What's done:
* Exclude detekt from `check` task
* Exclude linking release executable from ordinary CI workflow
* Enable gradle caches by default
* Disable redundant KMP targets compilation during build_and_test.yml
* Re-enable gradle-cache-action for build_and_test.yml

Author: petertrr

.github/workflows/build_and_test.yml

@@ -35,9 +35,15 @@ jobs:
           # additional files to calculate key for dependency cache
           gradle-dependencies-cache-key: |
             buildSrc/**/Versions.kt
-      # Until https://github.com/burrunan/gradle-cache-action/issues/42 is addressed, gradle should be run as a separate step
-      - name: Run gradle command
-        run: ./gradlew build --build-cache
+          # Note: https://github.com/burrunan/gradle-cache-action/issues/42 can possible break this
+          arguments: |
+            build
+            -PdisableRedundantTargets
+            -x linkReleaseExecutableLinuxX64
+            -x linkReleaseExecutableMingwX64
+            -x linkReleaseExecutableMacosX64
+          properties: |
+            org.gradle.caching=true
       - name: Upload gradle reports
         if: ${{ always() }}
         uses: actions/upload-artifact@v2

build.gradle.kts

@@ -4,6 +4,7 @@ import org.cqfn.save.buildutils.configurePublishing
 import org.cqfn.save.buildutils.configureVersioning
 import org.cqfn.save.buildutils.createDetektTask
 import org.cqfn.save.buildutils.createDiktatTask
+import org.cqfn.save.buildutils.disableRedundantKmpTargets
 import org.cqfn.save.buildutils.installGitHooks
 
 plugins {
@@ -26,6 +27,8 @@ allprojects {
     }
     configureDiktat()
     configureDetekt()
+
+    disableRedundantKmpTargets()
 }
 createDiktatTask()
 createDetektTask()

buildSrc/build.gradle.kts

@@ -8,7 +8,7 @@ repositories {
 
 dependencies {
     implementation("org.cqfn.diktat:diktat-gradle-plugin:1.0.0-rc.1")
-    implementation("io.gitlab.arturbosch.detekt:detekt-gradle-plugin:1.16.0")
+    implementation("io.gitlab.arturbosch.detekt:detekt-gradle-plugin:1.17.1")
     runtimeOnly(kotlin("gradle-plugin", "1.5.0"))
     implementation("io.github.gradle-nexus:publish-plugin:1.1.0")
     implementation("org.ajoberstar.reckon:reckon-gradle:0.13.0")

buildSrc/src/main/kotlin/org/cqfn/save/buildutils/DetektConfiguration.kt

@@ -8,6 +8,7 @@ import io.gitlab.arturbosch.detekt.Detekt
 import io.gitlab.arturbosch.detekt.DetektPlugin
 import io.gitlab.arturbosch.detekt.extensions.DetektExtension
 import org.gradle.api.Project
+import org.gradle.api.tasks.TaskProvider
 import org.gradle.kotlin.dsl.apply
 import org.gradle.kotlin.dsl.configure
 import org.gradle.kotlin.dsl.withType
@@ -21,6 +22,21 @@ fun Project.configureDetekt() {
         config = rootProject.files("detekt.yml")
         buildUponDefaultConfig = true
     }
+    // extremely awful hack to get to the point, when `DetektMultiplatform` has registered all the tasks
+    afterEvaluate {
+        // detekt registers tasks after Kotlin plugin has set up all targets
+        afterEvaluate {
+            // but they also use a nested `afterEvaluate` for interop with Android Gradle Plugin
+            afterEvaluate {
+                // so we need a third `afterEvaluate`, so that all detekt tasks are already registered
+                tasks.matching { it.name == "check" }.configureEach {
+                    dependsOn.removeIf {
+                        it is TaskProvider<*> && it.name.startsWith("detekt")
+                    }
+                }
+            }
+        }
+    }
 }
 
 /**

buildSrc/src/main/kotlin/org/cqfn/save/buildutils/KotlinConfiguration.kt

@@ -0,0 +1,32 @@
+/**
+ * Utility functions to configure Kotlin in Gradle
+ */
+
+package org.cqfn.save.buildutils
+
+import org.gradle.api.GradleException
+import org.gradle.api.Project
+import org.gradle.nativeplatform.platform.internal.DefaultNativePlatform
+
+/**
+ * Some Kotlin Multiplatform targets are available on multiple OS.
+ *
+ * @throws GradleException if executed on some exotic OS
+ */
+fun Project.disableRedundantKmpTargets() {
+    if (hasProperty("disableRedundantTargets") && (property("disableRedundantTargets") as String?) != "false") {
+        afterEvaluate {
+            val currentOs = DefaultNativePlatform.getCurrentOperatingSystem()
+            val redundantTarget: String? = when {
+                currentOs.isWindows -> "linuxX64"
+                currentOs.isMacOsX -> "linuxX64"
+                currentOs.isLinux -> null
+                else -> throw GradleException("Unknown operating system ${currentOs.name}")
+            }
+            tasks.matching { redundantTarget != null && it.name.contains(redundantTarget, ignoreCase = true) }
+                .configureEach {
+                    enabled = false
+                }
+        }
+    }
+}

gradle.properties

@@ -4,6 +4,7 @@ group=org.cqfn.save
 org.gradle.parallel=true
 org.gradle.vfs.watch=true
 org.gradle.jvmargs=-Xmx1g -XX:MaxMetaspaceSize=512m
+org.gradle.caching=true
 
 # Kotlin
 kotlin.code.style=official