Run tests parallelly on CI (#1787)

* Run tests parallelly on CI

Revert some changes in 2ba00cd to save more time on CI.

* Update CHANGELOG

* Configure jvmTarget in compilerOptions

---------

Co-authored-by: Paul Dingemans <pdingemans@bol.com>

.github/workflows/gradle-pr-build.yml

@@ -8,13 +8,14 @@ jobs:
     strategy:
       matrix:
         os: [ ubuntu-latest, windows-latest ]
+        jdk: [ 8, 11, 17, 19 ]
     runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@v3
       - uses: actions/setup-java@v3
         with:
           distribution: 'zulu'
-          java-version: 19
+          java-version: ${{ matrix.jdk }}
       - uses: gradle/gradle-build-action@v2
       - name: Build with release Kotlin version
         run: ./gradlew clean build ktlint --no-daemon

CHANGELOG.md

@@ -14,6 +14,8 @@ This project adheres to [Semantic Versioning](https://semver.org/).
 
 ### Changed
 
+* Revert to matrix build to speed up build, especially for the Windows related build ([#1786](https://github.com/pinterest/ktlint/pull/1787))
+
 ## [0.48.2] - 2023-01-21
 
 ### Additional clarification on API Changes in `0.48.0` and `0.48.1`

build-logic/src/main/kotlin/ktlint-kotlin-common.gradle.kts

@@ -1,14 +1,17 @@
+import org.jetbrains.kotlin.gradle.dsl.JvmTarget
+import org.jetbrains.kotlin.gradle.tasks.KotlinCompile
+
 plugins {
     kotlin("jvm")
 }
 
 kotlin {
-    jvmToolchain {
-        languageVersion.set(JavaLanguageVersion.of(8))
-    }
-
     // All modules, the CLI included, must have an explicit API
     explicitApi()
 }
 
-addAdditionalJdkVersionTests()
+tasks.withType<KotlinCompile>().configureEach {
+    compilerOptions {
+        jvmTarget.set(JvmTarget.JVM_1_8)
+    }
+}

build.gradle.kts

@@ -19,6 +19,20 @@ allprojects {
         } else {
             logger.warn("Skipping tests for task '$name' as system property 'skipTests=$skipTests'")
         }
+
+        val args = mutableSetOf<String>()
+        if (JavaVersion.current().isCompatibleWith(JavaVersion.VERSION_16)) {
+            // https://docs.gradle.org/7.5/userguide/upgrading_version_7.html#removes_implicit_add_opens_for_test_workers
+            args += listOf(
+                "--add-opens=java.base/java.lang=ALL-UNNAMED",
+                "--add-opens=java.base/java.util=ALL-UNNAMED",
+            )
+        }
+        if (JavaVersion.current().isCompatibleWith(JavaVersion.VERSION_18)) {
+            // https://openjdk.org/jeps/411
+            args += "-Djava.security.manager=allow"
+        }
+        jvmArgs(args)
     }
 }