Correctly set default for quarkus.flyway.validate-on-migrate

The default value for `quarkus.flyway.validate-on-migrate` was incorrectly set using a plain field default value. While not validated by Quarkus, this actually doesn't have any effect. What is important is setting `ConfigItem#defaultValue`.
fercomunello · Aug 31, 2022 · e0d548c · e0d548c
1 parent 7cfaf49
commit e0d548c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...flyway/runtime/src/main/java/io/quarkus/flyway/runtime/FlywayDataSourceRuntimeConfig.java b/...flyway/runtime/src/main/java/io/quarkus/flyway/runtime/FlywayDataSourceRuntimeConfig.java
@@ -17,7 +17,7 @@ public final class FlywayDataSourceRuntimeConfig {
      *
      * @return {@link FlywayDataSourceRuntimeConfig}
      */
-    public static final FlywayDataSourceRuntimeConfig defaultConfig() {
+    public static FlywayDataSourceRuntimeConfig defaultConfig() {
         return new FlywayDataSourceRuntimeConfig();
     }
 
@@ -126,7 +126,7 @@ public static final FlywayDataSourceRuntimeConfig defaultConfig() {
     /**
      * Whether to automatically call validate when performing a migration.
      */
-    @ConfigItem
+    @ConfigItem(defaultValue = "true")
     public boolean validateOnMigrate = true;
 
     /**