graalvm / setup-graalvm

GitHub Action for setting up GraalVM distributions.
https://www.graalvm.org
Universal Permissive License v1.0
192 stars 27 forks source link

Could not find GraalVM dev build #22

Closed alvarosanchez closed 1 year ago

alvarosanchez commented 1 year ago

The dev build is broken now:

image

fniephaus commented 1 year ago

We no longer provide JDK11-based dev builds. Please use JDK17 or pin the GraalVM version to 22.3.0.

This is the first time something like this happens. We'll for sure improve the error message. Technically, we could also fall back to the latest release that supports a certain Java version but I'm leaning towards that a hard failure is expected when you're building against a dev build. Otherwise, you wouldn't notice changes like this, right? Or should we fall back and print a big error message instead?

alvarosanchez commented 1 year ago

We have already removed Java 11 from the matrix, so not a big deal.

Agree that failing the build is correct. Perhaps if the message was Could not find GraalVM dev build for the requested java-version, it would have pointed me in the right direction. Also, updating README would have helped.