(causeway) branch main updated: CAUSEWAY-2445: remove -X build option (CI)

[ahuber]

This is an automated email from the ASF dual-hosted git repository.

ahuber pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/causeway.git


The following commit(s) were added to refs/heads/main by this push:
     new 7a86d199af2 CAUSEWAY-2445: remove -X build option (CI)
7a86d199af2 is described below

commit 7a86d199af2ee9105a498fb5767cdde60b4f214c
Author: Andi Huber <ahu...@apache.org>
AuthorDate: Wed May 28 04:56:27 2025 +0200

    CAUSEWAY-2445: remove -X build option (CI)
    
    - now that CAUSEWAY-3893 is fixed and CI build is green again
---
 .github/workflows/ci-build-artifacts-no-push-maven.yml |  1 -
 .../causeway/core/config/CausewayConfiguration.java    | 18 +++++++++---------
 2 files changed, 9 insertions(+), 10 deletions(-)

diff --git a/.github/workflows/ci-build-artifacts-no-push-maven.yml 
b/.github/workflows/ci-build-artifacts-no-push-maven.yml
index 09404562553..75162e85c94 100644
--- a/.github/workflows/ci-build-artifacts-no-push-maven.yml
+++ b/.github/workflows/ci-build-artifacts-no-push-maven.yml
@@ -131,7 +131,6 @@ jobs:
           -Dmaven.source.skip=true
           -Dproject.build.outputTimestamp=2025-01-01T00:00:00Z
           -e
-          -X
 
     # Autobuild attempts to build any compiled languages  (C/C++, C#, Go, or 
Java).
     # If this step fails, then you should remove it and run the build manually
diff --git 
a/core/config/src/main/java/org/apache/causeway/core/config/CausewayConfiguration.java
 
b/core/config/src/main/java/org/apache/causeway/core/config/CausewayConfiguration.java
index 7d41f7111cb..16ed2ce2b59 100644
--- 
a/core/config/src/main/java/org/apache/causeway/core/config/CausewayConfiguration.java
+++ 
b/core/config/src/main/java/org/apache/causeway/core/config/CausewayConfiguration.java
@@ -197,18 +197,18 @@ public Optional<String> valueOf(final String 
configurationPropertyName) {
      * The value of a specific configuration property
      *
      * @param configurationPropertyName  - eg as obtained from {@link 
#streamConfigurationPropertyNames()}.
-        * @param onError - callback in case the value retrieval thorws any 
exception
+     * @param onError - callback in case the value retrieval throws any 
exception
      */
     public Optional<String> valueOf(final String configurationPropertyName, 
final @Nullable Consumer<Throwable> onError) {
         try {
-                       return 
Optional.ofNullable(environment.getProperty(configurationPropertyName));
-               } catch (Throwable t) {
-                       if(onError!=null) {
-                               Try.run(()->onError.accept(t));
-                       }
-                       return Optional.empty();
-               }
-               
+            return 
Optional.ofNullable(environment.getProperty(configurationPropertyName));
+        } catch (Throwable t) {
+            if(onError!=null) {
+                Try.run(()->onError.accept(t));
+            }
+            return Optional.empty();
+        }
+        
     }
 
     private final Security security = new Security();