-
Notifications
You must be signed in to change notification settings - Fork 5
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
Update Java versions #151
Update Java versions #151
Conversation
rossabaker
commented
Jun 19, 2024
- Drops Java 8 on MacOS, which no longer exists. Fixes All PRs are failing on missing Temurin 8? #148.
- Bumps LTS version to 21. There's nothing special about 17 anymore.
ThisBuild / githubWorkflowJavaVersions := Seq(PrimaryJava, LTSJava) | ||
|
||
// MacOS runners do not have temurin@8 | ||
ThisBuild / githubWorkflowBuildMatrixExclusions := Seq( | ||
MatrixExclude(Map("os" -> MacOS, "java" -> JavaSpec.temurin("8").render)) |
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 could use the PrimaryJava
constant, but we really only need to drop what MacOS has dropped. Whenever PrimaryJava
progresses to 11, this can be removed... unless MacOS has already dropped that, too.
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.
Would it be reasonable to bump PrimaryJava
to 11 now? I don't have a lot of context but this is for new projects, and 8 seemed to be accelerating on it's way out
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.
I think we could bump to 11, but that can also happen in a follow up
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.
I asked Andrew about this and he proposed doing this bump in a follow-up just so it's more explicit haha thanks Andrew!
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.
Looks good, thanks for fixing this!