[core] uploader validation errors should be ignored in assembly/chang…

…elog mode
jreleaser · Feb 19, 2022 · 4936d2d · 4936d2d
1 parent 31bd512
commit 4936d2d
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 6 deletions.
diff --git a/core/jreleaser-model/src/main/java/org/jreleaser/model/validation/ArtifactoryValidator.java b/core/jreleaser-model/src/main/java/org/jreleaser/model/validation/ArtifactoryValidator.java
@@ -38,9 +38,10 @@ public static void validateArtifactory(JReleaserContext context, JReleaserContex
         for (Map.Entry<String, Artifactory> e : artifactory.entrySet()) {
             e.getValue().setName(e.getKey());
             if (!mode.validateConfig()) {
-                e.getValue().setActive(Active.NEVER);
+                validateArtifactory(context, mode, e.getValue(), new Errors());
+            } else {
+                validateArtifactory(context, mode, e.getValue(), errors);
             }
-            validateArtifactory(context, mode, e.getValue(), errors);
         }
     }
 

diff --git a/core/jreleaser-model/src/main/java/org/jreleaser/model/validation/HttpValidator.java b/core/jreleaser-model/src/main/java/org/jreleaser/model/validation/HttpValidator.java
@@ -41,9 +41,10 @@ public static void validateHttp(JReleaserContext context, JReleaserContext.Mode
         for (Map.Entry<String, Http> e : http.entrySet()) {
             e.getValue().setName(e.getKey());
             if (!mode.validateConfig()) {
-                e.getValue().setActive(Active.NEVER);
+                validateHttp(context, mode, e.getValue(), new Errors());
+            } else {
+                validateHttp(context, mode, e.getValue(), errors);
             }
-            validateHttp(context, mode, e.getValue(), errors);
         }
     }
 

diff --git a/core/jreleaser-model/src/main/java/org/jreleaser/model/validation/S3Validator.java b/core/jreleaser-model/src/main/java/org/jreleaser/model/validation/S3Validator.java
@@ -41,9 +41,10 @@ public static void validateS3(JReleaserContext context, JReleaserContext.Mode mo
         for (Map.Entry<String, S3> e : s3.entrySet()) {
             e.getValue().setName(e.getKey());
             if (!mode.validateConfig()) {
-                continue;
+                validateS3(context, mode, e.getValue(), new Errors());
+            } else {
+                validateS3(context, mode, e.getValue(), errors);
             }
-            validateS3(context, mode, e.getValue(), errors);
         }
     }