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

Add graalvm schema for org.fugerit.java:fj-doc-mod-opencsv #587

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fugerit79
Copy link
Contributor

What does this PR do?

I recently added to one of my project the native metadata needed for GraalVM :
https://github.com/fugerit-org/fj-doc/tree/main/fj-doc-mod-opencsv

As for the "CONTRIBUTING" section, the metadata are mantained and tested direcly in my project repository, But as I understand a pull request is needed to be listed here https://www.graalvm.org/native-image/libraries-and-frameworks/

PS: Thanks a lot for your great work with GraalVM, we converted already a few production projects to AOT and we are very happy for the gain in

Code sections where the PR accesses files, network, docker or some external service

Tests are note enabled as the metadata and testing is provided direclty on the prohect repository :

https://github.com/fugerit-org/fj-doc

A module test project using https://quarkus.io/ is included :
https://github.com/fugerit-org/fj-doc/tree/main/fj-doc-native-quarkus

And a workflow for continuous testing :
https://github.com/fugerit-org/fj-doc/actions/workflows/build_fj-doc-native-quarkus_test.yml

Checklist before merging

@fugerit79 fugerit79 requested a review from fniephaus as a code owner January 10, 2025 15:30
@fugerit79 fugerit79 force-pushed the 2-add-native-schema-support-for-orgfugeritjavafj-doc-mod-opencsv branch from 050be14 to a491bbe Compare January 10, 2025 16:13
@fugerit79
Copy link
Contributor Author

fugerit79 commented Jan 14, 2025

Good morning @fniephaus ,

This time the metadata are in a groupId/artifactId folder :

https://github.com/fugerit-org/fj-doc/tree/main/fj-doc-mod-opencsv/src/main/resources/META-INF/native-image/org.fugerit.java/fj-doc-mod-opencsv

We correted it for previous modules too.

Now the testing CI pipeline check graalvm 21 and 23 :

https://github.com/fugerit-org/fj-doc/blob/main/.github/workflows/build_fj-doc-native-quarkus_test.yml

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.

1 participant