clarify auto-detect skip vs skipModules

Author: hboutemy

src/main/java/org/apache/maven/plugins/artifact/buildinfo/AbstractBuildinfoMojo.java

@@ -160,8 +160,12 @@ public void execute() throws MojoExecutionException {
 
         if (!mono) {
             // if module skips install and/or deploy
-            if (isSkip(project)) {
-                getLog().info("Skipping goal because module skips install and/or deploy");
+            if (detectSkip && PluginUtil.isSkip(project)) {
+                getLog().info("Auto-skipping goal because module skips install and/or deploy");
+                return;
+            }
+            if (isSkipModule(project)) {
+                getLog().info("Skipping goal for module");
                 return;
             }
             // if multi-module build, generate (aggregate) buildinfo only in last module
@@ -384,20 +388,21 @@ private MavenProject getLastProject() {
     }
 
     protected boolean isSkip(MavenProject project) {
-        // manual/configured module skip
-        boolean skipModule = false;
-        if (skipModules != null && !skipModules.isEmpty()) {
-            if (skipModulesMatcher == null) {
-                FileSystem fs = FileSystems.getDefault();
-                skipModulesMatcher = skipModules.stream()
-                        .map(i -> fs.getPathMatcher("glob:" + i))
-                        .collect(Collectors.toList());
-            }
-            Path path = Paths.get(project.getGroupId() + '/' + project.getArtifactId());
-            skipModule = skipModulesMatcher.stream().anyMatch(m -> m.matches(path));
+        return isSkipModule(project) || (detectSkip && PluginUtil.isSkip(project));
+    }
+
+    protected boolean isSkipModule(MavenProject project) {
+        if (skipModules == null || skipModules.isEmpty()) {
+            return false;
+        }
+        if (skipModulesMatcher == null) {
+            FileSystem fs = FileSystems.getDefault();
+            skipModulesMatcher = skipModules.stream()
+                    .map(i -> fs.getPathMatcher("glob:" + i))
+                    .collect(Collectors.toList());
         }
-        // detected skip
-        return skipModule || (detectSkip && PluginUtil.isSkip(project));
+        Path path = Paths.get(project.getGroupId() + '/' + project.getArtifactId());
+        return skipModulesMatcher.stream().anyMatch(m -> m.matches(path));
     }
 
     private Toolchain getToolchain() {

src/main/java/org/apache/maven/plugins/artifact/buildinfo/DescribeBuildOutputMojo.java

@@ -122,7 +122,7 @@ private void describeBuildOutput() throws MojoExecutionException {
 
         for (MavenProject p : session.getProjects()) {
             boolean skipped = isSkip(p);
-            String s = skipped ? "not-deployed " : "             ";
+            String s = skipped ? (isSkipModule(p) ? "skipped      " : "not-deployed ") : "             ";
 
             // project = pom
             // detect Maven 4 consumer POM transient attachment