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

Fix error message when the JDK is not a GraalVM #616

Merged
merged 1 commit into from
Sep 3, 2024

Conversation

eregon
Copy link
Member

@eregon eregon commented Aug 30, 2024

Currently it still mentions gu but that's gone and a misleading error.

@eregon eregon requested a review from fniephaus August 30, 2024 14:36
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Aug 30, 2024
@eregon eregon force-pushed the fix-jdk-graalvm-error-message branch 3 times, most recently from 72797af to a6afd71 Compare August 30, 2024 14:48
@eregon
Copy link
Member Author

eregon commented Aug 30, 2024

I tried to fix the Gradle plugin in a similar manner.

@fniephaus fniephaus requested a review from dnestoro August 31, 2024 09:00
@eregon eregon force-pushed the fix-jdk-graalvm-error-message branch from a6afd71 to eb9a0e3 Compare September 2, 2024 13:03
@eregon eregon requested a review from fniephaus September 2, 2024 13:06
@dnestoro dnestoro force-pushed the fix-jdk-graalvm-error-message branch from eb9a0e3 to 1f6d43d Compare September 3, 2024 12:35
@eregon eregon requested a review from dnestoro September 3, 2024 16:30
@dnestoro dnestoro force-pushed the fix-jdk-graalvm-error-message branch from 1f6d43d to 81fe799 Compare September 3, 2024 16:34
@eregon eregon merged commit 2c8c2a7 into master Sep 3, 2024
221 checks passed
@eregon eregon deleted the fix-jdk-graalvm-error-message branch September 3, 2024 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants