-
Notifications
You must be signed in to change notification settings - Fork 19
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
Daikon handling in CI, cleanups #830
Conversation
.github/workflows/ci.yml
Outdated
# JDK 21 used by sanity before | ||
java: [{version: '8', experimental: false}, | ||
{version: '11', experimental: false}, | ||
{version: '17', experimental: false}, | ||
{version: '22', experimental: true}, | ||
{version: '22', experimental: false}, |
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.
This change means that the JDK 22 runtime is now also used to run the test cases - before, the JDK 21 runtime was used, as is the case for experimental platforms.
This reveals the failure with the non-junit tests, which is #826. So we either keep JDK 22 experimental or fix #826.
I can't reproduce the failures we get with :dataflow:busyExpressionTest
and the last few runs don't produce them. Not sure what is causing this.
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.
:dataflow:busyExpressionTest
fails again as soon as I remove the debugging for it in 56c9d40
No description provided.