garydgregory commented on pull request #208:
URL: https://github.com/apache/commons-io/pull/208#issuecomment-784550389
Hi @XenoAmess
-1 for several reasons:
Nits: The title of this PR ("fix ci error on jdk17") is misleading as (1)
the PR does not fix anything, it just ignores a part of the build and (2) this
is also for an Early Access Java build (jdk17-ea) and not a released Java
version of 17 which is not there yet.
Main: The 17-ea build is marked in our GitHub action as "experimental" and
is allowed to fail by definition. The whole point (in my mind at least) for
testing on EA builds is exactly what we are seeing now: a failure. In this
case, it is quite nice to see that JaCoCo is not ready for Java 17 yet; this
lets anyone who cares about the FOSS ecosystem go and help JaCoCo get up to
speed on Java 17. If by these efforts we can get a new release of JaCoCo out
sooner for us EA adventurers, then we are all better off.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
