From 003fae63aec3650b7e44e8b9606a76187351b533 Mon Sep 17 00:00:00 2001
From: Andrzej Jarmoniuk
Date: Sat, 19 Nov 2022 10:33:16 +0100
Subject: [PATCH] Make includeParent false by default to retain backward
compatibility.
---
.../codehaus/mojo/versions/DisplayPropertyUpdatesMojo.java | 4 ++--
.../org/codehaus/mojo/versions/PropertyUpdatesReportMojo.java | 4 ++--
.../mojo/versions/DisplayPropertyUpdatesMojoTest.java | 1 +
3 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/versions-maven-plugin/src/main/java/org/codehaus/mojo/versions/DisplayPropertyUpdatesMojo.java b/versions-maven-plugin/src/main/java/org/codehaus/mojo/versions/DisplayPropertyUpdatesMojo.java
index 65a183dc4..c39f11ac1 100644
--- a/versions-maven-plugin/src/main/java/org/codehaus/mojo/versions/DisplayPropertyUpdatesMojo.java
+++ b/versions-maven-plugin/src/main/java/org/codehaus/mojo/versions/DisplayPropertyUpdatesMojo.java
@@ -131,11 +131,11 @@ public class DisplayPropertyUpdatesMojo
private boolean allowIncrementalUpdates;
/**
- * Whether to include property updates from parent.
+ * Whether to include property updates from parent. Default: {@code false}
*
* @since 2.14.0
*/
- @Parameter( property = "includeParent", defaultValue = "true" )
+ @Parameter( property = "includeParent", defaultValue = "false" )
protected boolean includeParent = true;
// -------------------------- STATIC METHODS --------------------------
diff --git a/versions-maven-plugin/src/main/java/org/codehaus/mojo/versions/PropertyUpdatesReportMojo.java b/versions-maven-plugin/src/main/java/org/codehaus/mojo/versions/PropertyUpdatesReportMojo.java
index 4469b1da3..d106eb8fd 100644
--- a/versions-maven-plugin/src/main/java/org/codehaus/mojo/versions/PropertyUpdatesReportMojo.java
+++ b/versions-maven-plugin/src/main/java/org/codehaus/mojo/versions/PropertyUpdatesReportMojo.java
@@ -92,11 +92,11 @@ public class PropertyUpdatesReportMojo extends AbstractVersionsReportWhether to include property updates from parent.
+ * Whether to include property updates from parent. Default: {@code false}
*
* @since 2.14.0
*/
- @Parameter( property = "includeParent", defaultValue = "true" )
+ @Parameter( property = "includeParent", defaultValue = "false" )
private boolean includeParent = true;
/**
diff --git a/versions-maven-plugin/src/test/java/org/codehaus/mojo/versions/DisplayPropertyUpdatesMojoTest.java b/versions-maven-plugin/src/test/java/org/codehaus/mojo/versions/DisplayPropertyUpdatesMojoTest.java
index f306faed9..c9f896a11 100644
--- a/versions-maven-plugin/src/test/java/org/codehaus/mojo/versions/DisplayPropertyUpdatesMojoTest.java
+++ b/versions-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 ) ),