[CHORE](ci) Bump setup-java to v5 (Node 24) in publish-maven-to-codeartifact#31
Conversation
…rtifact Bumps actions/setup-java v4.8.0 -> v5.2.0 (Node 20 -> Node 24) in the publish-maven-to-codeartifact composite action, clearing the last residual Node 20 deprecation annotation found during overture-matchers #287 dev-publish testing. Inputs are unchanged/compatible, so it is a drop-in. Matches the consumer's own setup-java v5.2.0 pin (be666c2). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: John McCall <john@overturemaps.org>
Overture PRojection ReviewThis PR updates the ✅ Checks Passed
🚩 Flags
❓ Open Questions
No blocking issues found. The update is safe and aligns with org standards for CI maintenance. Recommend merge after confirming a successful publish run. |
|
Not in use yet, going to merge to iterate on downstream development. |
There was a problem hiding this comment.
Pull request overview
Updates the publish-maven-to-codeartifact composite action to use actions/setup-java v5 (Node 24) so the last remaining Node 20-based action dependency is removed ahead of GitHub’s Node 20 runtime cutoff.
Changes:
- Bump
actions/setup-javafrom v4.8.0 (c1e3236…, Node 20) to v5.2.0 (be666c2…, Node 24) in the JDK setup step.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
What
Bumps
actions/setup-javafromc1e323688fd81a25caa38c78aa6df2d33d3e20d9(v4.8.0, Node 20) tobe666c2fcd27ec809703dec50e508c2fdc7f6654(v5.2.0, Node 24) in thepublish-maven-to-codeartifactcomposite action.Why
This
setup-javastep was the last residual Node 20 source in the centralized CodeArtifact composite actions. A real dev-publish deploy during overture-matchers #287 testing surfaced a single Node 20 deprecation annotation traced to it. GitHub forces the Node 20 cutover on 2026-06-16.Compatibility
The step's inputs (
distribution: temurin,java-version-file,cache: maven,cache-dependency-path) are unchanged in setup-java v5 — v5's breaking changes are runner-version/runtime only, not these inputs. So this is a drop-in with nowith:changes. It also matches the consumer's own setup-java v5.2.0 pin (be666c2), which already runs warning-free.Validation
action.ymlparses (YAML safe_load OK).--persona=pedanticclean: no findings, 1 ignored (the pre-existing justifiedsetup-codeartifact@mainself-reference).git diffis exactly one line (thesetup-javauses:line).Closes #30
Part of #28
Part of OvertureMaps/ops-team#384
Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com