Make includeParent false by default to retain backward compatibility.

mojohaus · Nov 19, 2022 · 02d88b8 · 02d88b8
1 parent 8230434
commit 02d88b8
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/...ons-maven-plugin/src/main/java/org/codehaus/mojo/versions/DisplayPropertyUpdatesMojo.java b/...ons-maven-plugin/src/main/java/org/codehaus/mojo/versions/DisplayPropertyUpdatesMojo.java
@@ -131,11 +131,11 @@ public class DisplayPropertyUpdatesMojo
     private boolean allowIncrementalUpdates;
 
     /**
-     * <p>Whether to include property updates from parent.</p>
+     * <p>Whether to include property updates from parent. Default: {@code false}</p>
      *
      * @since 2.14.0
      */
-    @Parameter( property = "includeParent", defaultValue = "true" )
+    @Parameter( property = "includeParent", defaultValue = "false" )
     protected boolean includeParent = true;
 
     // -------------------------- STATIC METHODS --------------------------

diff --git a/...ions-maven-plugin/src/main/java/org/codehaus/mojo/versions/PropertyUpdatesReportMojo.java b/...ions-maven-plugin/src/main/java/org/codehaus/mojo/versions/PropertyUpdatesReportMojo.java
@@ -92,11 +92,11 @@ public class PropertyUpdatesReportMojo extends AbstractVersionsReport<PropertyUp
     private boolean autoLinkItems;
 
     /**
-     * <p>Whether to include property updates from parent.</p>
+     * <p>Whether to include property updates from parent. Default: {@code false}</p>
      *
      * @since 2.14.0
      */
-    @Parameter( property = "includeParent", defaultValue = "true" )
+    @Parameter( property = "includeParent", defaultValue = "false" )
     private boolean includeParent = true;
 
     /**

diff --git a/...maven-plugin/src/test/java/org/codehaus/mojo/versions/DisplayPropertyUpdatesMojoTest.java b/...maven-plugin/src/test/java/org/codehaus/mojo/versions/DisplayPropertyUpdatesMojoTest.java
@@ -81,6 +81,7 @@ public void testPropertiesFromParent() throws Exception
         mojo.outputFile = tempFile.toFile();
         mojo.setPluginContext( new HashMap<>() );
         mojo.artifactMetadataSource = MockUtils.mockArtifactMetadataSource();
+        mojo.includeParent = true;
         mojo.execute();
 
         assertThat( String.join( "", Files.readAllLines( tempFile ) ),