Skip to content

Simplify JDK matrix: - #5988

Open
cpovirk wants to merge 3 commits into
masterfrom
matrix-reloaded
Open

Simplify JDK matrix:#5988
cpovirk wants to merge 3 commits into
masterfrom
matrix-reloaded

Conversation

@cpovirk

@cpovirk cpovirk commented Jul 31, 2026

Copy link
Copy Markdown
Member
  • We don't need to list "JDK 25 on ubuntu-latest" in include because it's covered by the generated os-java cross product.
  • We can add 23 to the java list so that "JDK 23 on ubuntu-latest" is generated automatically instead of having to list it in include.

- We don't need to list "JDK 25 on ubuntu-latest" in `include` because it's covered by the generated `os`-`java` cross product.
- We can add 23 to the `java` list so that "JDK 23 on ubuntu-latest" is generated automatically instead of having to list it in `include`.
@cushon

cushon commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Thanks. I'm trying to remember why we even have 23, it may have been to navigate breaking changes in 23 before JDK 25 was released. At this point it might make sense to just have the two latest LTSs and EA, or maybe also the latest STS. I don't expect a lot of usage of JDK 23 now that there's a newer LTS.

@cpovirk

cpovirk commented Aug 2, 2026

Copy link
Copy Markdown
Member Author

That is pretty much exactly how I have looked at it for Guava. So naturally I'm here because of a test that fails under JDK 23 :) But that's unlikely to happen with any frequency, and also it doesn't at all invalidate your point that zero end users will be affected.

Whatever we do here with 23, we should reconcile it with the Copybara config that I changed in cl/957613778 but lacked permissions to put into effect.

The other failing check here will be addressed by #5987, which is mostly about pinning actions by hash. After that, we should tell Dependabot to recreate #5990 so that it will use hashes for it.

@cpovirk cpovirk mentioned this pull request Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants