Skip to content
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

Java 21 update #3213

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Java 21 update #3213

wants to merge 1 commit into from

Conversation

iluwatar
Copy link
Owner

Pull Request Template

What does this PR do?

Update project to support JDK 21

Copy link

github-actions bot commented Mar 20, 2025

PR Summary

This PR updates the project to support JDK 21. Changes include updating the pom.xml file to use Java 21, and modifying the Github Actions workflows (maven-ci.yml and maven-pr-builder.yml) to use JDK 21 for building and testing.

Changes

File Summary
.github/workflows/maven-ci.yml Updated the Java CI workflow to use JDK 21 instead of JDK 17. This includes changing the setup-java action and the runner to use Ubuntu 22.04.
.github/workflows/maven-pr-builder.yml Updated the Java PR builder workflow to use JDK 21 instead of JDK 17. This includes changing the setup-java action and the runner to use Ubuntu 22.04.
pom.xml Updated the pom.xml to use Java 21 and updated the maven-compiler-plugin version to 3.14.0. Also updated the Lombok version to 1.18.36 and added annotationProcessorPaths for Lombok.

autogenerated by presubmit.ai

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Review Summary

Commits Considered (1)
  • 83410f4: update pom.xml and github actions scripts
Files Processed (3)
  • .github/workflows/maven-ci.yml (2 hunks)
  • .github/workflows/maven-pr-builder.yml (2 hunks)
  • pom.xml (3 hunks)
Actionable Comments (0)
Skipped Comments (0)

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.

1 participant