stevedlawrence commented on a change in pull request #652:
URL: https://github.com/apache/daffodil/pull/652#discussion_r725149551
##########
File path: .github/workflows/main.yml
##########
@@ -34,7 +34,7 @@ jobs:
fail-fast: false
matrix:
distribution: [ temurin ]
- java_version: [ 8, 11, 16 ]
+ java_version: [ 8, 11, 17 ]
scala_version: [ 2.12.15 ]
Review comment:
Any suggestions for what to say? I'm not exactly sure what makes sense.
The releases include the version of Scala and other dependencies that we
require, so we'd have to warn people that switching dependencies other than
what we released with might break something. But that seems unnecessary--that
usually goes without saying.
We could also warn that using old releases with newer version of Java might
break, but I feel like that's just a known risk of running the latest and
greatest of things. And according to that bug, Scala 2.12.13 doesn't have this
particular issue, so our Daffodil 3.1.0 release shouldn't be affected.
--
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]
