Skip to content

fix: reintroduce support for GraalVM 17 #1010

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

Merged
merged 8 commits into from
Apr 8, 2025

Conversation

diegomarquezp
Copy link
Contributor

@diegomarquezp diegomarquezp commented Apr 4, 2025

Context doc

TL;DR: we are adding another Cloud Build job to test against GraalVM for JDK 17.

The failing linter checks will be solved in the next release cycle. They were introduced in #1003

@product-auto-label product-auto-label bot added the size: m Pull request size is medium. label Apr 4, 2025
@diegomarquezp
Copy link
Contributor Author

/gcbrun

@diegomarquezp diegomarquezp marked this pull request as ready for review April 8, 2025 19:54
@diegomarquezp diegomarquezp requested a review from a team as a code owner April 8, 2025 19:54
@diegomarquezp diegomarquezp requested a review from mpeddada1 April 8, 2025 20:24
Copy link
Contributor

@mpeddada1 mpeddada1 left a comment

Choose a reason for hiding this comment

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

Looks great!

@diegomarquezp diegomarquezp merged commit 1906261 into main Apr 8, 2025
54 of 58 checks passed
@diegomarquezp diegomarquezp deleted the introduce-graalvm-image-c-support branch April 8, 2025 21:05
lqiu96 pushed a commit that referenced this pull request May 1, 2025
* fix: reintroduce support for GraalVM 17

* correct coment

* fix copyright year

* non image-dependent steps to not depend on image building step

* restore step dependency

* checkout fix branch for testing

* clone with branch

* use main branch of sdk-platform
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants