(maven-site-plugin) 01/01: [MSITE-987] Consistently log when skipping report goal execution

[michaelo]

This is an automated email from the ASF dual-hosted git repository.

michaelo pushed a commit to branch MSITE-987
in repository https://gitbox.apache.org/repos/asf/maven-site-plugin.git

commit 8ceb7b02a892ea4084bb1788e6dfdba896521c9c
Author: Michael Osipov <micha...@apache.org>
AuthorDate: Sun Oct 29 19:11:07 2023 +0100

    [MSITE-987] Consistently log when skipping report goal execution
    
    This closes #162
---
 .../apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java   | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java
 
b/src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java
index 76bcfac5..90f60165 100644
--- 
a/src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java
+++ 
b/src/main/java/org/apache/maven/plugins/site/render/AbstractSiteRenderingMojo.java
@@ -225,12 +225,14 @@ public abstract class AbstractSiteRenderingMojo extends 
AbstractSiteDescriptorMo
         // filter out reports that can't be generated
         List<MavenReportExecution> reportExecutions = new 
ArrayList<>(allReports.size());
         for (MavenReportExecution exec : allReports) {
+            String reportMojoInfo = exec.getPlugin().getId() + ":" + 
exec.getGoal();
             try {
                 if (exec.canGenerateReport()) {
                     reportExecutions.add(exec);
+                } else if (exec.isUserDefined()) {
+                    getLog().info("Skipping " + reportMojoInfo + " report");
                 }
             } catch (MavenReportException e) {
-                String reportMojoInfo = exec.getPlugin().getId() + ":" + 
exec.getGoal();
                 throw new MojoExecutionException(
                         "Failed to determine whether report '" + 
reportMojoInfo + "' can be generated", e);
             }