efegokdemir opened a new pull request, #562:
URL: https://github.com/apache/maven-ear-plugin/pull/562

   ## Summary
   
   `initializeJbossConfiguration` now treats omitted optional JBoss 
configuration elements as absent values instead of dereferencing null children. 
This addresses #513 without changing the generated configuration for supplied 
elements.
   
   ## Changes
   
   - Safely read optional child values and attributes, including loader 
repository settings.
   - Added a regression test covering a minimal JBoss configuration with 
omitted optional children.
   
   ## Testing
   
   - `mvn -B -ntp -DskipITs -Dspotless.check.skip=true 
-Dspotless.apply.skip=true test` — passed (49 tests).
   - `mvn -B -ntp -DskipITs -Dspotless.check.skip=true 
-Dspotless.apply.skip=true verify` — passed.
   - `git diff --check` — passed.
   - `mvn -B -ntp -Prun-its -Dspotless.check.skip=true 
-Dspotless.apply.skip=true verify` — blocked by the local Java 8/Groovy 
environment: the Invoker tests report `Unsupported class file major version 
71`; the failure occurs before the scenario assertions.
   
   AI assistance was used during analysis and implementation; the patch and 
test results were reviewed before submission.
   
   Fixes #513


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to