Property to disable tasks not needed for plugin's main functionality

This property is a "quick" solution to make this feature available
and gather feedback from users. Eventually, tasks should be:
- moved to a separate plugin
- or removed completely if not considered useful anymore

This would be a breaking change and thus should be done only in the next
major release.

Author: jjohannes

src/main/java/org/gradlex/javamodule/dependencies/JavaModuleDependenciesPlugin.java

@@ -68,6 +68,7 @@
 public abstract class JavaModuleDependenciesPlugin implements Plugin<ExtensionAware> {
 
     private static final String EXTRA_JAVA_MODULE_INFO_PLUGIN_ID = "org.gradlex.extra-java-module-info";
+    private static final String REGISTER_HELP_TASKS_PROPERTY = "org.gradlex.java-module-dependencies.register-help-tasks";
 
     @Override
     public void apply(ExtensionAware projectOrSettings) {
@@ -92,6 +93,10 @@ private void applyProject(Project project) {
     }
 
     private void setupForJavaProject(Project project, JavaModuleDependenciesExtension javaModuleDependencies) {
+        boolean registerHelpTasks = Boolean.parseBoolean(project.getProviders()
+                .gradleProperty(REGISTER_HELP_TASKS_PROPERTY)
+                .getOrElse("true"));
+
         SourceSetContainer sourceSets = project.getExtensions().getByType(SourceSetContainer.class);
         sourceSets.all(sourceSet -> {
             process(REQUIRES, sourceSet.getImplementationConfigurationName(), sourceSet, project, javaModuleDependencies);
@@ -103,21 +108,22 @@ private void setupForJavaProject(Project project, JavaModuleDependenciesExtensio
             javaModuleDependencies.doAddRequiresRuntimeSupport(sourceSet, sourceSet);
         });
 
-        TaskProvider<Task> checkAllModuleInfo = project.getTasks().register("checkAllModuleInfo", t -> {
+        setupDirectivesDSL(project, javaModuleDependencies);
+
+        TaskProvider<Task> checkAllModuleInfo = registerHelpTasks ? project.getTasks().register("checkAllModuleInfo", t -> {
             t.setGroup(VERIFICATION_GROUP);
             t.setDescription("Check scope and order of directives in 'module-info.java' files");
-        });
+        }) : null;
 
-        setupDirectivesDSL(project, javaModuleDependencies);
-
-        setupOrderingCheckTasks(project, checkAllModuleInfo, javaModuleDependencies);
-        setupModuleDependenciesTask(project);
-        setupReportTasks(project, javaModuleDependencies);
-        setupMigrationTasks(project, javaModuleDependencies);
+        if (registerHelpTasks) {
+            setupOrderingCheckTasks(project, checkAllModuleInfo, javaModuleDependencies);
+            setupModuleDependenciesTask(project);
+            setupReportTasks(project, javaModuleDependencies);
+            setupMigrationTasks(project, javaModuleDependencies);
+        }
 
-        project.getPlugins().withId("com.autonomousapps.dependency-analysis", analysisPlugin -> {
-            DependencyAnalysisBridge.registerDependencyAnalysisPostProcessingTask(project, checkAllModuleInfo);
-        });
+        project.getPlugins().withId("com.autonomousapps.dependency-analysis", analysisPlugin ->
+                DependencyAnalysisBridge.registerDependencyAnalysisPostProcessingTask(project, checkAllModuleInfo));
     }
 
     private void setupExtraJavaModulePluginBridge(Project project, JavaModuleDependenciesExtension javaModuleDependencies) {
@@ -143,17 +149,15 @@ private void setupDirectivesDSL(Project project, JavaModuleDependenciesExtension
     private void setupModuleDependenciesTask(Project project) {
         SourceSetContainer sourceSets = project.getExtensions().getByType(SourceSetContainer.class);
         TaskProvider<ModuleDependencyReport> moduleDependencies = project.getTasks().register("moduleDependencies", ModuleDependencyReport.class, t -> t.setGroup(HELP_GROUP));
-        sourceSets.all(sourceSet -> {
-            moduleDependencies.configure(t -> {
-                HashSet<Configuration> joined = new HashSet<>();
-                if (t.getConfigurations() != null) {
-                    joined.addAll(t.getConfigurations());
-                }
-                joined.add(project.getConfigurations().getByName(sourceSet.getCompileClasspathConfigurationName()));
-                joined.add(project.getConfigurations().getByName(sourceSet.getRuntimeClasspathConfigurationName()));
-                t.setConfigurations(joined);
-            });
-        });
+        sourceSets.all(sourceSet -> moduleDependencies.configure(t -> {
+            HashSet<Configuration> joined = new HashSet<>();
+            if (t.getConfigurations() != null) {
+                joined.addAll(t.getConfigurations());
+            }
+            joined.add(project.getConfigurations().getByName(sourceSet.getCompileClasspathConfigurationName()));
+            joined.add(project.getConfigurations().getByName(sourceSet.getRuntimeClasspathConfigurationName()));
+            t.setConfigurations(joined);
+        }));
     }
 
     private void setupReportTasks(Project project, JavaModuleDependenciesExtension javaModuleDependencies) {

src/main/java/org/gradlex/javamodule/dependencies/internal/bridges/DependencyAnalysisBridge.java

@@ -26,11 +26,12 @@
 import org.gradlex.javamodule.dependencies.tasks.ModuleDirectivesOrderingCheck;
 import org.gradlex.javamodule.dependencies.tasks.ModuleDirectivesScopeCheck;
 
+import javax.annotation.Nullable;
 import java.io.File;
 
 public class DependencyAnalysisBridge {
 
-    public static void registerDependencyAnalysisPostProcessingTask(Project project, TaskProvider<Task> checkAllModuleInfo) {
+    public static void registerDependencyAnalysisPostProcessingTask(Project project, @Nullable TaskProvider<Task> checkAllModuleInfo) {
         TaskContainer tasks = project.getTasks();
         SourceSetContainer sourceSets = project.getExtensions().getByType(SourceSetContainer.class);
 
@@ -54,7 +55,9 @@ public static void registerDependencyAnalysisPostProcessingTask(Project project,
         project.getExtensions().getByType(AbstractExtension.class)
                 .registerPostProcessingTask(checkModuleDirectivesScope);
 
-        checkAllModuleInfo.configure(t -> t.dependsOn(checkModuleDirectivesScope));
+        if (checkAllModuleInfo != null) {
+            checkAllModuleInfo.configure(t -> t.dependsOn(checkModuleDirectivesScope));
+        }
         tasks.withType(ModuleDirectivesOrderingCheck.class).configureEach(t -> t.mustRunAfter(checkModuleDirectivesScope));
     }
 }