Update ktfmt to 0.58 (#5885)

Newer Ktfmt versions require Java 17+, so I disabled some sensitive
Ktfmt tests, when these run on lower JVM versions.

Pull request: #5885

Author: lefou

libs/kotlinlib/test/src/mill/kotlinlib/contrib/ktfmt/KtfmtModuleTests.scala

@@ -25,37 +25,45 @@ object KtfmtModuleTests extends TestSuite {
     }
 
     test("ktfmt - kotlin style") {
-      assert(
-        checkState(
-          afterFormat(before, style = "kotlin"),
-          after / "style/kotlin"
+      if (!scala.util.Properties.isJavaAtLeast(17)) "Ktfmt requires Java 17"
+      else
+        assert(
+          checkState(
+            afterFormat(before, style = "kotlin"),
+            after / "style/kotlin"
+          )
         )
-      )
     }
 
     test("ktfmt - google style") {
-      assert(
-        checkState(
-          afterFormat(before, style = "google"),
-          after / "style/google"
+      if (!scala.util.Properties.isJavaAtLeast(17)) "Ktfmt requires Java 17"
+      else
+        assert(
+          checkState(
+            afterFormat(before, style = "google"),
+            after / "style/google"
+          )
         )
-      )
     }
 
     test("ktfmt - meta style") {
-      assert(
-        checkState(
-          afterFormat(before, style = "meta"),
-          after / "style/meta"
+      if (!scala.util.Properties.isJavaAtLeast(17)) "Ktfmt requires Java 17"
+      else
+        assert(
+          checkState(
+            afterFormat(before, style = "meta"),
+            after / "style/meta"
+          )
         )
-      )
     }
 
     test("ktfmt - dry-run") {
-      checkState(
-        afterFormat(before, format = true),
-        before
-      )
+      if (!scala.util.Properties.isJavaAtLeast(17)) "Ktfmt requires Java 17"
+      else
+        checkState(
+          afterFormat(before, format = true),
+          before
+        )
     }
 
     test("ktfmt - don't remove unused imports") {
@@ -73,13 +81,14 @@ object KtfmtModuleTests extends TestSuite {
     }
 
     test("formatAll") {
-
-      assert(
-        checkState(
-          afterFormatAll(before),
-          after / "style/kotlin"
+      if (!scala.util.Properties.isJavaAtLeast(17)) "Ktfmt requires Java 17"
+      else
+        assert(
+          checkState(
+            afterFormatAll(before),
+            after / "style/kotlin"
+          )
         )
-      )
     }
   }
 

mill-build/src/millbuild/Deps.scala

@@ -242,7 +242,7 @@ object Deps {
     val kotlinxHtmlJvm = mvn"org.jetbrains.kotlinx:kotlinx-html:0.11.0"
     val koverCli = mvn"org.jetbrains.kotlinx:kover-cli:$koverVersion"
     val koverJvmAgent = mvn"org.jetbrains.kotlinx:kover-jvm-agent:$koverVersion"
-    val ktfmt = mvn"com.facebook:ktfmt:0.53"
+    val ktfmt = mvn"com.facebook:ktfmt:0.58"
     val ktlint = mvn"com.pinterest.ktlint:ktlint-core:0.49.1"
     val palantirFormat = mvn"com.palantir.javaformat:palantir-java-format:2.74.0"
     val proguard = mvn"com.guardsquare:proguard-base:7.7.0"