stevedlawrence commented on code in PR #1236:
URL: https://github.com/apache/daffodil/pull/1236#discussion_r1595508282


##########
.github/workflows/main.yml:
##########
@@ -39,7 +39,7 @@ jobs:
       fail-fast: false
       matrix:
         java_distribution: [ temurin ]
-        java_version: [ 8, 11, 17 ]
+        java_version: [ 8, 11, 17, 21 ]

Review Comment:
   Looks like something is broken on Windows on Java 21. Some CLI tests are 
possibly hanging. Tests don't normally take this long.
   
   As to what to drop, I would prefer we keep 11 and 17 since they are LTS and 
we claim to support them. I think the plan is to drop Java 8 support for 
Daffodil 4.0.0, so we'll be back to the normal 3 versions of java when that 
happens.



-- 
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