Sonar code smells fixes

altro3 · altro3 · commit fe3f83d9cb14 · 2023-12-03T15:19:44.000+07:00
diff --git a/aot-plugin/src/main/java/io/micronaut/gradle/aot/AbstractMicronautAotCliTask.java b/aot-plugin/src/main/java/io/micronaut/gradle/aot/AbstractMicronautAotCliTask.java
@@ -127,7 +127,7 @@ public final void execute() throws IOException {
             }
         } finally {
             onSuccess(outputDir);
-            argFile.delete();
+            Files.delete(argFile.toPath());
         }
     }
 
diff --git a/aot-plugin/src/main/java/io/micronaut/gradle/aot/AotShadowSupport.java b/aot-plugin/src/main/java/io/micronaut/gradle/aot/AotShadowSupport.java
@@ -33,6 +33,9 @@
  */
 class AotShadowSupport {
 
+    private AotShadowSupport() {
+    }
+
     static void registerShadowJar(Project project,
                                   ArchiveOperations archiveOperations,
                                   TaskContainer tasks,
diff --git a/aot-plugin/src/main/java/io/micronaut/gradle/aot/MicronautAOTConfigWriterTask.java b/aot-plugin/src/main/java/io/micronaut/gradle/aot/MicronautAOTConfigWriterTask.java
@@ -77,23 +77,21 @@ public abstract class MicronautAOTConfigWriterTask extends DefaultTask {
     private static void booleanOptimization(Properties props, String optimizationId, Provider<Boolean> provider) {
         if (provider.isPresent()) {
             String key = optimizationId + ".enabled";
-            if (!props.containsKey(key)) {
-                props.put(key, String.valueOf(provider.get()));
-            }
+            props.computeIfAbsent(key, (k) -> String.valueOf(provider.get()));
         }
     }
 
     private static void stringListParameter(Properties props, String parameter, ListProperty<String> provider) {
         if (provider.isPresent()) {
             List<String> elements = provider.get();
-            if (!props.containsKey(parameter) && !elements.isEmpty()) {
-                props.put(parameter, String.join(",", elements));
+            if (!elements.isEmpty()) {
+                props.computeIfAbsent(parameter, (k) -> String.join(",", elements));
             }
         }
     }
     private static void stringParameter(Properties props, String parameter, Property<String> provider) {
         if (provider.isPresent()) {
-            props.put(parameter, provider.get());
+            props.setProperty(parameter, provider.get());
         }
     }
 
@@ -112,10 +110,10 @@ void writeConfigFile() {
             props.putAll(optimizations.getConfigurationProperties().get());
         }
         if (!props.containsKey(KnownMissingTypesSourceGenerator.OPTION.key())) {
-            props.put(KnownMissingTypesSourceGenerator.OPTION.key(), String.join(",", MicronautAotPlugin.TYPES_TO_CHECK));
+            props.setProperty(KnownMissingTypesSourceGenerator.OPTION.key(), String.join(",", MicronautAotPlugin.TYPES_TO_CHECK));
         }
         if (!props.containsKey(AbstractStaticServiceLoaderSourceGenerator.SERVICE_TYPES)) {
-            props.put(AbstractStaticServiceLoaderSourceGenerator.SERVICE_TYPES, String.join(",", MicronautAotPlugin.SERVICE_TYPES));
+            props.setProperty(AbstractStaticServiceLoaderSourceGenerator.SERVICE_TYPES, String.join(",", MicronautAotPlugin.SERVICE_TYPES));
         }
         booleanOptimization(props, GraalVMOptimizationFeatureSourceGenerator.ID, getForNative());
         booleanOptimization(props, LogbackConfigurationSourceGenerator.ID, optimizations.getReplaceLogbackXml());
diff --git a/aot-plugin/src/main/java/io/micronaut/gradle/aot/MicronautAotPlugin.java b/aot-plugin/src/main/java/io/micronaut/gradle/aot/MicronautAotPlugin.java
@@ -160,7 +160,7 @@ private void registerPrepareOptimizationsTasks(Project project, Configurations c
     }
 
     private void registerCreateSamplesTasks(Project project, Configuration optimizerRuntimeClasspath, Configuration applicationClasspath, TaskContainer tasks, AOTExtension aotExtension) {
-        TaskProvider<Task> createAotSampleConfigurationFiles = tasks.register("createAotSampleConfigurationFiles", task -> 
+        TaskProvider<Task> createAotSampleConfigurationFiles = tasks.register("createAotSampleConfigurationFiles", task ->
             task.setDescription("Generates Micronaut AOT sample configuration files")
         );
         for (OptimizerIO.TargetRuntime targetRuntime : OptimizerIO.TargetRuntime.values()) {
@@ -325,8 +325,8 @@ private void registerJavaExecOptimizedRun(Project project,
                     public void execute(Task t) {
                         if (task.getLogger().isDebugEnabled()) {
                             task.getLogger().debug(
-                                    "Running optimized entry point: " + task.getMainClass().get() +
-                                            "\nClasspath:\n    " + task.getClasspath().getFiles()
+                                    "Running optimized entry point: {}\nClasspath:\n    {}",
+                                    task.getMainClass().get(), task.getClasspath().getFiles()
                                             .stream()
                                             .map(File::getName)
                                             .collect(Collectors.joining("\n    "))
diff --git a/aot-plugin/src/main/java/io/micronaut/gradle/aot/MicronautAotSampleConfTask.java b/aot-plugin/src/main/java/io/micronaut/gradle/aot/MicronautAotSampleConfTask.java
@@ -33,7 +33,7 @@ protected void configureExtraArguments(List<String> args) {
     protected void onSuccess(File outputDir) {
         File sampleFile = new File(outputDir, getTargetRuntime().map(runtime -> runtime.getSimpleName() + ".properties").orElse("sample.properties").get());
         if (sampleFile.exists()) {
-            System.out.println("Sample configuration file written to " + Strings.clickableUrl(sampleFile));
+            getLogger().info("Sample configuration file written to {}", Strings.clickableUrl(sampleFile));
         }
     }
 }
diff --git a/buildSrc/src/main/groovy/io.micronaut.internal.build.functional-testing.gradle b/buildSrc/src/main/groovy/io.micronaut.internal.build.functional-testing.gradle
@@ -32,7 +32,7 @@ extensions.create("functionalTesting", FunctionalTestingExtension)
 
 def prepareRepository = tasks.register("prepareRepository", Sync) {
     from configurations.pluginsUnderTest
-    into file("${buildDir}/aggregatedRepo")
+    into layout.buildDirectory.dir("aggregatedRepo").get().asFile
 }
 
 tasks.withType(Test).configureEach { Test test ->
diff --git a/buildSrc/src/main/groovy/io/micronaut/internal/build/sourcegen/SimpleSourceProcessor.java b/buildSrc/src/main/groovy/io/micronaut/internal/build/sourcegen/SimpleSourceProcessor.java
@@ -28,7 +28,6 @@
 import org.gradle.api.tasks.TaskAction;
 
 import java.io.IOException;
-import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.util.Map;
diff --git a/crac-plugin/src/main/java/io/micronaut/gradle/crac/CRaCConfiguration.java b/crac-plugin/src/main/java/io/micronaut/gradle/crac/CRaCConfiguration.java
@@ -37,7 +37,7 @@ public interface CRaCConfiguration {
 
     /**
      * The platform to specify in the FROM instruction defaults to {@value MicronautCRaCPlugin#CRAC_DEFAULT_BASE_IMAGE_PLATFORM}.
-     * @return the platform (can be removed with {@code platform.convention(null)} in the {@link CRaCConfiguration} extension)
+     * @return the platform (can be removed with {@code platform.convention(null)} in the CRaCConfiguration extension)
      * @deprecated use {@link #getArch()} instead
      */
     @SuppressWarnings("java:S6355") // We need Java 8... Java 8 doesn't have forRemoval and since
diff --git a/docker-plugin/src/main/java/io/micronaut/gradle/docker/MicronautDockerPlugin.java b/docker-plugin/src/main/java/io/micronaut/gradle/docker/MicronautDockerPlugin.java
@@ -310,7 +310,7 @@ private TaskProvider<NativeImageDockerfile> configureNativeDockerBuild(Project p
                     task.doLast(new Action<>() {
                         @Override
                         public void execute(Task task1) {
-                            System.out.println("AWS Lambda ZIP built: " + lambdaZip.get());
+                            project.getLogger().info("AWS Lambda ZIP built: {}", lambdaZip.get());
                         }
                     });
                     task.finalizedBy(removeContainer);
diff --git a/docker-plugin/src/main/java/io/micronaut/gradle/docker/MicronautDockerfile.java b/docker-plugin/src/main/java/io/micronaut/gradle/docker/MicronautDockerfile.java
@@ -80,7 +80,7 @@ public void create() throws IOException {
         if (getDockerfileTweaks().isPresent()) {
             DockerfileEditor.apply(getObjects(), this, getDockerfileTweaks().get());
         }
-        System.out.println("Dockerfile written to: " + getDestFile().get().getAsFile().getAbsolutePath());
+        getLogger().info("Dockerfile written to: {}", getDestFile().get().getAsFile().getAbsolutePath());
     }
 
     protected void setupInstructions(List<Instruction> additionalInstructions) {
diff --git a/docker-plugin/src/main/java/io/micronaut/gradle/docker/NativeImageDockerfile.java b/docker-plugin/src/main/java/io/micronaut/gradle/docker/NativeImageDockerfile.java
@@ -376,7 +376,7 @@ public DirectoryProperty getPgoProfilesDirectory() {
             @Override
             public void execute(Task task) {
                 java.io.File f = getDestFile().get().getAsFile();
-                System.out.println("Dockerfile written to: " + f.getAbsolutePath());
+                getLogger().info("Dockerfile written to: {}", f.getAbsolutePath());
             }
         });
     }
diff --git a/graalvm-plugin/src/main/java/io/micronaut/gradle/graalvm/GraalUtil.java b/graalvm-plugin/src/main/java/io/micronaut/gradle/graalvm/GraalUtil.java
@@ -7,6 +7,9 @@
  */
 public final class GraalUtil {
 
+    private GraalUtil() {
+    }
+
     /**
      * @return Return whether the JVM in use a GraalVM JVM.
      */
diff --git a/graalvm-plugin/src/main/java/io/micronaut/gradle/graalvm/MicronautGraalPlugin.java b/graalvm-plugin/src/main/java/io/micronaut/gradle/graalvm/MicronautGraalPlugin.java
@@ -85,7 +85,7 @@ public void apply(Project project) {
                 }
         );
         TaskContainer tasks = project.getTasks();
-        project.getPluginManager().withPlugin("application", plugin -> {
+        project.getPluginManager().withPlugin("application", plugin ->
             tasks.withType(BuildNativeImageTask.class).named("nativeCompile", nativeImageTask -> {
                 MicronautRuntime mr = PluginsHelper.resolveRuntime(project);
                 if (mr.isLambdaProvided()) {
@@ -98,8 +98,7 @@ public void apply(Project project) {
                         nativeImageTask.getOptions().get().getMainClass().set(nativeLambdaExtension.getLambdaRuntimeClassName());
                     }
                 }
-            });
-        });
+        }));
     }
 
     private void workaroundForResourcesDirectoryMissing(Project project) {
diff --git a/minimal-plugin/src/main/java/io/micronaut/gradle/LenientGradle.java b/minimal-plugin/src/main/java/io/micronaut/gradle/LenientGradle.java
@@ -26,6 +26,9 @@
 public class LenientGradle {
     private static final Map<String, Optional<Class<?>>> CLASSES = new ConcurrentHashMap<>();
 
+    private LenientGradle() {
+    }
+
     private static Optional<Class<?>> findClass(String name) {
         return CLASSES.computeIfAbsent(name, LenientGradle::loadClass);
     }
diff --git a/minimal-plugin/src/main/java/io/micronaut/gradle/MicronautComponentPlugin.java b/minimal-plugin/src/main/java/io/micronaut/gradle/MicronautComponentPlugin.java
@@ -31,7 +31,6 @@
 import org.gradle.api.tasks.testing.Test;
 
 import java.io.File;
-import java.util.Arrays;
 import java.util.Collections;
 import java.util.HashSet;
 import java.util.List;
diff --git a/minimal-plugin/src/main/java/io/micronaut/gradle/PluginsHelper.java b/minimal-plugin/src/main/java/io/micronaut/gradle/PluginsHelper.java
@@ -90,6 +90,9 @@ public abstract class PluginsHelper {
             VALIDATION_VERSION_PROPERTY
     );
 
+    private PluginsHelper() {
+    }
+
     public static void maybeAddMicronautPlaformBom(Project p, Configuration configuration) {
         MicronautExtension micronautExtension = p.getExtensions().findByType(MicronautExtension.class);
         configuration.getDependencies().addAllLater(
diff --git a/minimal-plugin/src/main/java/io/micronaut/gradle/ShadowPluginSupport.java b/minimal-plugin/src/main/java/io/micronaut/gradle/ShadowPluginSupport.java
@@ -21,6 +21,9 @@
 public class ShadowPluginSupport {
     public static final String SHADOW_PLUGIN = "com.github.johnrengelman.shadow";
 
+    private ShadowPluginSupport() {
+    }
+
     public static void withShadowPlugin(Project p, Runnable action) {
         p.getPluginManager().withPlugin(SHADOW_PLUGIN, unused -> action.run());
     }
diff --git a/minimal-plugin/src/main/java/io/micronaut/gradle/Strings.java b/minimal-plugin/src/main/java/io/micronaut/gradle/Strings.java
@@ -22,8 +22,12 @@
 import static java.util.Locale.ENGLISH;
 
 public abstract class Strings {
+
+    private Strings() {
+    }
+
     public static String capitalize(String name) {
-        if (name == null || name.length() == 0) {
+        if (name == null || name.isEmpty()) {
             return name;
         }
         return name.substring(0, 1).toUpperCase(ENGLISH) + name.substring(1);
diff --git a/minimal-plugin/src/main/java/io/micronaut/gradle/graalvm/GraalUtil.java b/minimal-plugin/src/main/java/io/micronaut/gradle/graalvm/GraalUtil.java
@@ -7,6 +7,9 @@
  */
 public final class GraalUtil {
 
+    private GraalUtil() {
+    }
+
     /**
      * @return Return whether the JVM in use a GraalVM JVM.
      */
diff --git a/openapi-plugin/src/main/java/io/micronaut/gradle/openapi/tasks/OpenApiClientWorkAction.java b/openapi-plugin/src/main/java/io/micronaut/gradle/openapi/tasks/OpenApiClientWorkAction.java
@@ -18,7 +18,6 @@
 import java.util.List;
 
 import io.micronaut.openapi.generator.MicronautCodeGeneratorBuilder;
-import io.micronaut.openapi.generator.MicronautCodeGeneratorOptionsBuilder.GeneratorLanguage;
 
 import org.gradle.api.provider.ListProperty;
 import org.gradle.api.provider.Property;
diff --git a/openapi-plugin/src/main/java/io/micronaut/gradle/openapi/tasks/OpenApiServerWorkAction.java b/openapi-plugin/src/main/java/io/micronaut/gradle/openapi/tasks/OpenApiServerWorkAction.java
@@ -16,7 +16,6 @@
 package io.micronaut.gradle.openapi.tasks;
 
 import io.micronaut.openapi.generator.MicronautCodeGeneratorBuilder;
-import io.micronaut.openapi.generator.MicronautCodeGeneratorOptionsBuilder.GeneratorLanguage;
 
 import org.gradle.api.provider.Property;
 
diff --git a/platform-catalog-plugin/src/main/java/io/micronaut/gradle/catalog/LenientVersionCatalogParser.java b/platform-catalog-plugin/src/main/java/io/micronaut/gradle/catalog/LenientVersionCatalogParser.java
@@ -117,8 +117,8 @@ private static Boolean expectBoolean(TomlTable table, String element) {
     private void parseLibrary(String alias, TomlTable librariesTable, RichVersionParser strictVersionParser) {
         Object gav = librariesTable.get(alias);
         TomlPosition position = librariesTable.inputPositionOf(alias);
-        if (gav instanceof String) {
-            List<String> splitted = splitToList((String) gav);
+        if (gav instanceof String gavStr) {
+            List<String> splitted = splitToList(gavStr);
             if (splitted.size() == 3) {
                 String group = splitted.get(0);
                 String name = splitted.get(1);
@@ -147,9 +147,8 @@ private void parseLibrary(String alias, TomlTable librariesTable, RichVersionPar
             }
         }
         VersionModel versionModel = null;
-        if (version instanceof String) {
-            String require = (String) version;
-            RichVersion richVersion = strictVersionParser.parse(require);
+        if (version instanceof String requireVersion) {
+            RichVersion richVersion = strictVersionParser.parse(requireVersion);
             versionModel = new VersionModel(null, richVersion, position);
         } else if (version instanceof TomlTable versionTable) {
             String versionRef = versionTable.getString("ref");
@@ -182,8 +181,8 @@ private void parseVersion(String alias, TomlTable versionsTable, RichVersionPars
         Boolean rejectAll = null;
         Object version = versionsTable.get(alias);
         TomlPosition position = versionsTable.inputPositionOf(alias);
-        if (version instanceof String) {
-            require = (String) version;
+        if (version instanceof String versionStr) {
+            require = versionStr;
             RichVersion richVersion = strictVersionParser.parse(require);
             model.addVersion(new VersionModel(alias, richVersion, position));
         } else if (version instanceof TomlTable versionTable) {
diff --git a/platform-catalog-plugin/src/main/java/io/micronaut/gradle/catalog/MicronautCatalogSettingsPlugin.java b/platform-catalog-plugin/src/main/java/io/micronaut/gradle/catalog/MicronautCatalogSettingsPlugin.java
@@ -136,8 +136,8 @@ private Provider<String> readFromVersionCatalog(Settings settings) {
                         TomlTable versions = toml.getTable("versions");
                         if (versions != null) {
                             Object micronaut = versions.get("micronaut");
-                            if (micronaut instanceof String) {
-                                return (String) micronaut;
+                            if (micronaut instanceof String micronautVer) {
+                                return micronautVer;
                             }
                         }
                         return null;
diff --git a/platform-catalog-plugin/src/main/java/io/micronaut/gradle/catalog/VersionCatalogTomlModel.java b/platform-catalog-plugin/src/main/java/io/micronaut/gradle/catalog/VersionCatalogTomlModel.java
@@ -49,7 +49,9 @@ public void addLibrary(Library library) {
     }
 
     public void addVersion(VersionModel version) {
-        assert version.getReference() != null && version.getVersion() != null;
+        if (version.getReference() == null || version.getVersion() == null) {
+            throw new IllegalArgumentException("version.getReference() is null or version.getVersion() is null");
+        }
         versions.add(version);
         versionAliasToVersion.put(version.getReference(), version);
     }
diff --git a/test-resources-plugin/src/main/java/io/micronaut/gradle/testresources/MicronautTestResourcesPlugin.java b/test-resources-plugin/src/main/java/io/micronaut/gradle/testresources/MicronautTestResourcesPlugin.java
@@ -334,7 +334,7 @@ static ArrayList<Integer> parseVersion(String testedVersion) {
         }
         return Arrays.stream(version.split("\\."))
                 .map(String::trim)
-                .map(s -> s.replaceAll("[^0-9]", ""))
+                .map(s -> s.replaceAll("\\D", ""))
                 .map(Integer::parseInt)
                 .collect(Collectors.toCollection(ArrayList::new));
     }
diff --git a/test-resources-plugin/src/main/java/io/micronaut/gradle/testresources/internal/TestResourcesGraalVM.java b/test-resources-plugin/src/main/java/io/micronaut/gradle/testresources/internal/TestResourcesGraalVM.java
@@ -29,18 +29,20 @@
  * issues.
  */
 public final class TestResourcesGraalVM {
+
     public static final String ENABLED_PROPERTY_NAME = "testresources.native";
 
+    private TestResourcesGraalVM() {
+    }
+
     public static void configure(Project project,
                                  Configuration client,
                                  TaskProvider<StartTestResourcesService> internalStart) {
         GraalVMExtension graalVMExtension = project.getExtensions().findByType(GraalVMExtension.class);
-        graalVMExtension.getBinaries().all(b -> {
-            b.getRuntimeArgs().addAll(internalStart.map(task -> {
-                ServerConnectionParametersProvider provider = new ServerConnectionParametersProvider(task.getSettingsDirectory());
-                return provider.asArguments();
-            }));
-        });
+        graalVMExtension.getBinaries().all(b -> b.getRuntimeArgs().addAll(internalStart.map(task -> {
+            ServerConnectionParametersProvider provider = new ServerConnectionParametersProvider(task.getSettingsDirectory());
+            return provider.asArguments();
+        })));
         ProviderFactory providers = project.getProviders();
         boolean includeClient = Boolean.TRUE.equals(providers
                 .systemProperty(ENABLED_PROPERTY_NAME)

Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,7 @@ public final void execute() throws IOException {`
`127`	`127`	`}`
`128`	`128`	`} finally {`
`129`	`129`	`onSuccess(outputDir);`
`130`		`- argFile.delete();`
	`130`	`+ Files.delete(argFile.toPath());`
`131`	`131`	`}`
`132`	`132`	`}`
`133`	`133`
Original file line number	Diff line number	Diff line change
`@@ -77,23 +77,21 @@ public abstract class MicronautAOTConfigWriterTask extends DefaultTask {`
`77`	`77`	`private static void booleanOptimization(Properties props, String optimizationId, Provider<Boolean> provider) {`
`78`	`78`	`if (provider.isPresent()) {`
`79`	`79`	`String key = optimizationId + ".enabled";`
`80`		`- if (!props.containsKey(key)) {`
`81`		`- props.put(key, String.valueOf(provider.get()));`
`82`		`- }`
	`80`	`+ props.computeIfAbsent(key, (k) -> String.valueOf(provider.get()));`
`83`	`81`	`}`
`84`	`82`	`}`
`85`	`83`
`86`	`84`	`private static void stringListParameter(Properties props, String parameter, ListProperty<String> provider) {`
`87`	`85`	`if (provider.isPresent()) {`
`88`	`86`	`List<String> elements = provider.get();`
`89`		`- if (!props.containsKey(parameter) && !elements.isEmpty()) {`
`90`		`- props.put(parameter, String.join(",", elements));`
	`87`	`+ if (!elements.isEmpty()) {`
	`88`	`+ props.computeIfAbsent(parameter, (k) -> String.join(",", elements));`
`91`	`89`	`}`
`92`	`90`	`}`
`93`	`91`	`}`
`94`	`92`	`private static void stringParameter(Properties props, String parameter, Property<String> provider) {`
`95`	`93`	`if (provider.isPresent()) {`
`96`		`- props.put(parameter, provider.get());`
	`94`	`+ props.setProperty(parameter, provider.get());`
`97`	`95`	`}`
`98`	`96`	`}`
`99`	`97`
`@@ -112,10 +110,10 @@ void writeConfigFile() {`
`112`	`110`	`props.putAll(optimizations.getConfigurationProperties().get());`
`113`	`111`	`}`
`114`	`112`	`if (!props.containsKey(KnownMissingTypesSourceGenerator.OPTION.key())) {`
`115`		`- props.put(KnownMissingTypesSourceGenerator.OPTION.key(), String.join(",", MicronautAotPlugin.TYPES_TO_CHECK));`
	`113`	`+ props.setProperty(KnownMissingTypesSourceGenerator.OPTION.key(), String.join(",", MicronautAotPlugin.TYPES_TO_CHECK));`
`116`	`114`	`}`
`117`	`115`	`if (!props.containsKey(AbstractStaticServiceLoaderSourceGenerator.SERVICE_TYPES)) {`
`118`		`- props.put(AbstractStaticServiceLoaderSourceGenerator.SERVICE_TYPES, String.join(",", MicronautAotPlugin.SERVICE_TYPES));`
	`116`	`+ props.setProperty(AbstractStaticServiceLoaderSourceGenerator.SERVICE_TYPES, String.join(",", MicronautAotPlugin.SERVICE_TYPES));`
`119`	`117`	`}`
`120`	`118`	`booleanOptimization(props, GraalVMOptimizationFeatureSourceGenerator.ID, getForNative());`
`121`	`119`	`booleanOptimization(props, LogbackConfigurationSourceGenerator.ID, optimizations.getReplaceLogbackXml());`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ protected void configureExtraArguments(List<String> args) {`
`33`	`33`	`protected void onSuccess(File outputDir) {`
`34`	`34`	`File sampleFile = new File(outputDir, getTargetRuntime().map(runtime -> runtime.getSimpleName() + ".properties").orElse("sample.properties").get());`
`35`	`35`	`if (sampleFile.exists()) {`
`36`		`- System.out.println("Sample configuration file written to " + Strings.clickableUrl(sampleFile));`
	`36`	`+ getLogger().info("Sample configuration file written to {}", Strings.clickableUrl(sampleFile));`
`37`	`37`	`}`
`38`	`38`	`}`
`39`	`39`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ extensions.create("functionalTesting", FunctionalTestingExtension)`
`32`	`32`
`33`	`33`	`def prepareRepository = tasks.register("prepareRepository", Sync) {`
`34`	`34`	`from configurations.pluginsUnderTest`
`35`		`- into file("${buildDir}/aggregatedRepo")`
	`35`	`+ into layout.buildDirectory.dir("aggregatedRepo").get().asFile`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`tasks.withType(Test).configureEach { Test test ->`
Original file line number	Diff line number	Diff line change
`@@ -310,7 +310,7 @@ private TaskProvider<NativeImageDockerfile> configureNativeDockerBuild(Project p`
`310`	`310`	`task.doLast(new Action<>() {`
`311`	`311`	`@Override`
`312`	`312`	`public void execute(Task task1) {`
`313`		`- System.out.println("AWS Lambda ZIP built: " + lambdaZip.get());`
	`313`	`+ project.getLogger().info("AWS Lambda ZIP built: {}", lambdaZip.get());`
`314`	`314`	`}`
`315`	`315`	`});`
`316`	`316`	`task.finalizedBy(removeContainer);`
Original file line number	Diff line number	Diff line change
`@@ -80,7 +80,7 @@ public void create() throws IOException {`
`80`	`80`	`if (getDockerfileTweaks().isPresent()) {`
`81`	`81`	`DockerfileEditor.apply(getObjects(), this, getDockerfileTweaks().get());`
`82`	`82`	`}`
`83`		`- System.out.println("Dockerfile written to: " + getDestFile().get().getAsFile().getAbsolutePath());`
	`83`	`+ getLogger().info("Dockerfile written to: {}", getDestFile().get().getAsFile().getAbsolutePath());`
`84`	`84`	`}`
`85`	`85`
`86`	`86`	`protected void setupInstructions(List<Instruction> additionalInstructions) {`
Original file line number	Diff line number	Diff line change
`@@ -376,7 +376,7 @@ public DirectoryProperty getPgoProfilesDirectory() {`
`376`	`376`	`@Override`
`377`	`377`	`public void execute(Task task) {`
`378`	`378`	`java.io.File f = getDestFile().get().getAsFile();`
`379`		`- System.out.println("Dockerfile written to: " + f.getAbsolutePath());`
	`379`	`+ getLogger().info("Dockerfile written to: {}", f.getAbsolutePath());`
`380`	`380`	`}`
`381`	`381`	`});`
`382`	`382`	`}`
Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ public void apply(Project project) {`
`85`	`85`	`}`
`86`	`86`	`);`
`87`	`87`	`TaskContainer tasks = project.getTasks();`
`88`		`- project.getPluginManager().withPlugin("application", plugin -> {`
	`88`	`+ project.getPluginManager().withPlugin("application", plugin ->`
`89`	`89`	`tasks.withType(BuildNativeImageTask.class).named("nativeCompile", nativeImageTask -> {`
`90`	`90`	`MicronautRuntime mr = PluginsHelper.resolveRuntime(project);`
`91`	`91`	`if (mr.isLambdaProvided()) {`
`@@ -98,8 +98,7 @@ public void apply(Project project) {`
`98`	`98`	`nativeImageTask.getOptions().get().getMainClass().set(nativeLambdaExtension.getLambdaRuntimeClassName());`
`99`	`99`	`}`
`100`	`100`	`}`
`101`		`- });`
`102`		`- });`
	`101`	`+ }));`
`103`	`102`	`}`
`104`	`103`
`105`	`104`	`private void workaroundForResourcesDirectoryMissing(Project project) {`
Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,9 @@`
`26`	`26`	`public class LenientGradle {`
`27`	`27`	`private static final Map<String, Optional<Class<?>>> CLASSES = new ConcurrentHashMap<>();`
`28`	`28`
	`29`	`+ private LenientGradle() {`
	`30`	`+ }`
	`31`	`+`
`29`	`32`	`private static Optional<Class<?>> findClass(String name) {`
`30`	`33`	`return CLASSES.computeIfAbsent(name, LenientGradle::loadClass);`
`31`	`34`	`}`
Original file line number	Diff line number	Diff line change
`@@ -21,6 +21,9 @@`
`21`	`21`	`public class ShadowPluginSupport {`
`22`	`22`	`public static final String SHADOW_PLUGIN = "com.github.johnrengelman.shadow";`
`23`	`23`
	`24`	`+ private ShadowPluginSupport() {`
	`25`	`+ }`
	`26`	`+`
`24`	`27`	`public static void withShadowPlugin(Project p, Runnable action) {`
`25`	`28`	`p.getPluginManager().withPlugin(SHADOW_PLUGIN, unused -> action.run());`
`26`	`29`	`}`