[MPLUGIN-511] improvements based on feedback

apache · Mar 14, 2024 · 4607b85 · 4607b85
1 parent 58648cf
commit 4607b85
Showing 1 changed file with 11 additions and 5 deletions.
diff --git a/...lugin-report-plugin/src/main/java/org/apache/maven/plugin/plugin/report/PluginReport.java b/...lugin-report-plugin/src/main/java/org/apache/maven/plugin/plugin/report/PluginReport.java
@@ -110,8 +110,13 @@ public class PluginReport extends AbstractMavenReport {
     @Parameter
     private List<RequirementsHistory> requirementsHistories = new ArrayList<>();
 
+    /**
+     * Plugin's version range for automatic detection of requirements history.
+     *
+     * @since 3.12.0
+     */
     @Parameter(defaultValue = "[0,)")
-    private String detectRequirementsHistory;
+    private String requirementsHistoryDetectionRange;
 
     @Component
     private RuntimeInformation rtInfo;
@@ -183,8 +188,8 @@ protected void executeReport(Locale locale) throws MavenReportException {
             // detect requirements history
             String v = null;
             try {
-                List<Version> versions = discoverVersions(detectRequirementsHistory);
-                getLog().info("Detecting requirements history for " + detectRequirementsHistory + ": "
+                List<Version> versions = discoverVersions(requirementsHistoryDetectionRange);
+                getLog().info("Detecting requirements history for " + requirementsHistoryDetectionRange + ": "
                         + versions.size());
 
                 Collections.reverse(versions);
@@ -193,10 +198,11 @@ protected void executeReport(Locale locale) throws MavenReportException {
                     MavenProject versionProject = buildMavenProject(v);
                     RequirementsHistory requirements = RequirementsHistory.discoverRequirements(versionProject);
                     requirementsHistories.add(requirements);
-                    getLog().info("- " + requirements);
+                    getLog().info("  - " + requirements);
                 }
             } catch (VersionRangeResolutionException vrre) {
-                throw new MavenReportException("Cannot resolve past versions " + detectRequirementsHistory, vrre);
+                throw new MavenReportException(
+                        "Cannot resolve past versions " + requirementsHistoryDetectionRange, vrre);
             } catch (ProjectBuildingException pbe) {
                 throw new MavenReportException("Cannot resolve MavenProject for version " + v, pbe);
             }