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 Native Image and Truffle compilation sections for GraalPy guides #1563

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

steve-s
Copy link

@steve-s steve-s commented Dec 11, 2024

From GraalVM JDK 23, Micronaut apps with GraalPy can be also compiled with GraalVM Native Image.

We also noticed that this guide is broken for some reason: https://guides.micronaut.io/latest/micronaut-graalpy-python-package.html. Any ideas why that could be?

@steve-s
Copy link
Author

steve-s commented Dec 11, 2024

/CC @sdelamo, @olyagpl I don't seem to have a permission to request a review?

@olyagpl
Copy link
Contributor

olyagpl commented Dec 12, 2024

@steve-s, I joined as a reviewer, left comments, but can't tell why this guide is broken. Sergio should know better.

@olyagpl
Copy link
Contributor

olyagpl commented Dec 14, 2024

@steve-s, could you please, in your PR, remove this extra 1 in the NI Maven plugin code snippet:
Screenshot 2024-12-14 at 12 08 44
I accidentally left it.

Co-authored-by: Olga Gupalo <olga.gupalo@oracle.com>
src/docs/common/snippets/common-graalpy-graalvm.adoc Outdated Show resolved Hide resolved
src/docs/common/snippets/common-graalpy-graalvm.adoc Outdated Show resolved Hide resolved
@steve-s
Copy link
Author

steve-s commented Dec 18, 2024

@sdelamo can you please take a look and eventually merge

@@ -64,7 +64,7 @@ It is possible to customize the name of the native executable or pass additional
<version>0.10.3</version>
<configuration>
<!-- <1> -->
<imageName>mn-graalvm-application</imageName> <1>
<imageName>mn-graalvm-application</imageName>
Copy link
Contributor

Choose a reason for hiding this comment

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

Thank you!

@sdelamo sdelamo self-requested a review December 19, 2024 14:07
@steve-s
Copy link
Author

steve-s commented Jan 8, 2025

ping :-)

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.

2 participants