-
Notifications
You must be signed in to change notification settings - Fork 881
Fix graalvm compatibility #7160
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
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7160 +/- ##
=========================================
Coverage 89.85% 89.85%
Complexity 6622 6622
=========================================
Files 740 740
Lines 20007 20007
Branches 1968 1968
=========================================
Hits 17978 17978
Misses 1439 1439
Partials 590 590 ☔ View full report in Codecov by Sentry. |
cc @jeanbisutti |
Hmm.. its not clear to me what the problem is with AttributeType that requires it to be handled with this special case. Can you provide a repro of the issue? Would good to add it to our graal test suite. |
@ericdallo Could you please provide the GraalVM version for which the problem appears? |
@jack-berg I reproed that using the log appender of timbre here on clojure-lsp, I can't spend more time on this though, so after this PR, I included on clojure-lsp here to fix my case meanwhile |
@jack-berg The native tests are executed with GraalVM 21:
|
See #7167 |
This fix a graalvm compatibility with the logger api (only one I tested in my lib), this exports the graalvm config automatically for clients.