Fix some bugs

lukebemishprojects · Dec 10, 2024 · 4afbd81 · 4afbd81
1 parent fdc699a
commit 4afbd81
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/main/java/dev/lukebemish/immaculate/steps/WrapperFormattingStep.java b/src/main/java/dev/lukebemish/immaculate/steps/WrapperFormattingStep.java
@@ -29,7 +29,6 @@ public WrapperFormattingStep(String name, String workflowName, Project project,
             }
             return null;
         }).flatMap(javaToolchainService::launcherFor).orElse(javaToolchainService.launcherFor(spec -> {})));
-        getJavaLauncher().convention(javaToolchainService.launcherFor(spec -> {}));
         this.objectFactory = objectFactory;
         getHideStacktrace().convention(project.getGradle().getStartParameter().getShowStacktrace() == ShowStacktrace.INTERNAL_EXCEPTIONS);
         this.getDependencies().getRuntime().add("dev.lukebemish.immaculate:wrapper", dep -> {

diff --git a/...main/java/dev/lukebemish/immaculate/wrapper/googlejavaformat/GoogleJavaFormatWrapper.java b/...main/java/dev/lukebemish/immaculate/wrapper/googlejavaformat/GoogleJavaFormatWrapper.java
@@ -29,17 +29,18 @@ public String format(String fileName, String text) {
         try {
             int ok = formatter.format(fullArgs);
             if (ok != 0) {
-                throw new RuntimeException("Failed to format " + fileName);
+                throw new RuntimeException("Failed to format " + fileName + ", exit code: " + ok);
             }
         } catch (Exception e) {
             if (e instanceof RuntimeException runtimeException) {
                 throw runtimeException;
             } else {
                 throw new RuntimeException(e);
             }
+        } finally {
+            errWriter.flush();
+            outWriter.flush();
         }
-        errWriter.flush();
-        outWriter.flush();
         return outputStream.toString(StandardCharsets.UTF_8);
     }
 }