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

#267 GraalVM Reachability Metadata #268

Merged
merged 2 commits into from
Jan 27, 2025

Conversation

iromu
Copy link
Contributor

@iromu iromu commented Jan 26, 2025

About the changes

Closes #267

@coveralls
Copy link
Collaborator

Coverage Status

coverage: 86.587%. remained the same
when pulling 4a0e2fa on iromu:feature/graalvm-reachability-metadata
into eaff6f4 on Unleash:main.

@sighphyre
Copy link
Member

I know very little about actually using Graal with Java but is it possible to include an example or a test for this? The reason I ask is because in the near future this library is going to take on a native dependency and I suspect that's going to break this.

This does look fine to me though

Copy link
Member

@chriswk chriswk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution @iromu .

@sighphyre sighphyre merged commit 30389f8 into Unleash:main Jan 27, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

graalvm reachability metadata
4 participants