ScalafmtReflect: don't wrap in UnknownError

This leads to a confusing "unknown error" message while masking the real problem.
scalameta · Apr 26, 2022 · 9840207 · 9840207
1 parent 7196593
commit 9840207
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/scalafmt-dynamic/src/main/scala/org/scalafmt/dynamic/ScalafmtReflect.scala b/scalafmt-dynamic/src/main/scala/org/scalafmt/dynamic/ScalafmtReflect.scala
@@ -58,7 +58,7 @@ case class ScalafmtReflect(
 
   private def parseConfigWith(
       f: => Try[Object]
-  )(g: Throwable => ScalafmtDynamicError): Try[ScalafmtReflectConfig] =
+  )(g: Throwable => Throwable): Try[ScalafmtReflectConfig] =
     f.map { configured =>
       new ScalafmtReflectConfig(this)(configured.invoke("get"))
     }.recoverWith { case ReflectionException(e) => Failure(g(e)) }
@@ -69,9 +69,7 @@ case class ScalafmtReflect(
     }
 
   def parseConfigFromString(text: String): Try[ScalafmtReflectConfig] =
-    parseConfigWith(parseConfigPre300(text))(
-      ScalafmtDynamicError.UnknownError.apply
-    )
+    parseConfigWith(parseConfigPre300(text))(identity)
 
   private def parseConfigPost300(path: Path): Try[Object] = {
     if (version < ScalafmtVersion(3, 0, 0, 7)) parseConfigPre300(path)