Skip to content

feat(serialization): Add GraalVM metadata configuration #1905

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

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

Conversation

jreijn
Copy link
Contributor

@jreijn jreijn commented Jun 19, 2025

Issue #, if available: #1839

Description of changes:

This PR changed:

  • Added GraalVM metadata generation for powertools-serialization
  • Moved existing example to powertools-serialization-sam
  • Introduced new example for powertools-serialization-sam-graalvm
  • Moved GraalVM metadata for tests to test package

Breaking change checklist

RFC issue #:

  • Migration process documented
  • Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@jreijn jreijn force-pushed the feat/graalvm-metadata-serialization branch from d5356e3 to 4c9956d Compare June 23, 2025 13:03
Copy link

@phipag
Copy link
Contributor

phipag commented Jun 25, 2025

Hey @jreijn, is this PR ready for review?

@jreijn
Copy link
Contributor Author

jreijn commented Jun 25, 2025

Hi @phipag, yes I believe this PR is ready for review.

@phipag
Copy link
Contributor

phipag commented Jun 25, 2025

Awesome, thanks again for your work here @jrein. As you can see, there are a couple of pending PRs at the moment that I need to review but I'll do my best to test this soon.

@jreijn
Copy link
Contributor Author

jreijn commented Jun 25, 2025

Thanks @phipag. Don't worry, I understand. I'm still working on powertools-batch, so I will just keep filling your backlog 😉

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

Successfully merging this pull request may close these issues.

2 participants