-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Upgrade to GraalVM 20.3.1 #14457
Upgrade to GraalVM 20.3.1 #14457
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a small gripe.
...ions/tika/runtime/src/main/java/io/quarkus/tika/graalvm/CleanerNotSupportedSubstitution.java
Outdated
Show resolved
Hide resolved
CI looks good (except for "MicroProfile TCKs Tests" which timed out and AFAIU is irrelevant to the GraalVM update) : https://github.com/zakkak/quarkus/actions/runs/499615927 |
Let's promote it then! |
Merged! Nice work @zakkak . I think we can work on moving master to 21.0 now, will you do it? |
Thanks
Sure :) |
Replaces #14455
Copying @gsmet's original PR body:
Reverts changes made to work around issues introduced by 20.3.0 that are no longer present in 20.3.1 (see #13644).
More specifically it reverts: