Skip to content
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

Issue725 read me supported version syntax #750

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

skateball
Copy link
Contributor

Description:
some small changes on README.md to enhance readabilty

Related issue:
fixes #725

Check list:

  • [no] Mark if documentation changes are required.
  • [no] Mark if tests were added or updated to cover the changes.

@skateball skateball requested a review from a team as a code owner February 11, 2025 20:57
@skateball
Copy link
Contributor Author

Hi @fniephaus
did a small change in a markdown file and afterwards the test https://github.com/actions/setup-java/actions/runs/13272353142/job/37054560743?pr=750 runs in an error because downloading https://download.oracle.com/graalvm/18/latest/graalvm-jdk-18_macos-x64_bin.tar.gz runs in a 404

We're sorry, the page you requested was not found.
We have recorded this error (404) to help us fix the problem.
You may wish to try again using one of the tools below.

Can you please check?
Regards Christian

@fniephaus
Copy link
Contributor

Hi @skateball,

I'm not sure why this is failing but this seems to be a transient issue. GraalVM for JDK 18 indeed does not exist and the test is supposed to check for exactly that. Instead of an exception being thrown, however, I see this in the log:

Received promise resolved instead of rejected

I'd assume this is unrelated to your markdown changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support all Java versions
2 participants