-
-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update usage instructions to remove Java setup #9
Comments
Thank you for pointing it out. That's definitely correct but ubuntu-22.04, which was ubuntu-latest until a few months ago, uses Java 11 by default, even though Jena5 requires Java 17+. There should be documentation on how to change the Java version, as some users are unaware of the correspondence between the runners and tool versions. |
Indeed, but I saw that there is an input for the Jena version:
So on older runners the instructions could further specify how to fetch an older version of Jena compatible with the Java version on that runner. |
Doesn't make sense. Why limit the Jena version when setup-java will solve the problem? |
You are right, now that I re-read my comment it is confusing. You already clarified what I was trying to get at:
|
Currently the README instructions mention the following:
This setup for Java is redundant and therefore not required as the Ubuntu runners come with Java by default. See actions/runner-images#9848 (default changed from 11 to 17 from 22.04 to 24.04) and https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2004-Readme.md (it appears even 20.04 had 11 as default).
Thanks for this maintaining this Action!
The text was updated successfully, but these errors were encountered: