Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

dockerfile: update to use java 21 #517

Closed
wants to merge 2 commits into from
Closed

dockerfile: update to use java 21 #517

wants to merge 2 commits into from

Conversation

chenrui333
Copy link
Contributor

similar to #494

@fwcd
Copy link
Owner

fwcd commented Oct 11, 2023

Thanks, we'll have to migrate to Gradle 8.4 first though (PR is underway)

@chenrui333
Copy link
Contributor Author

Thanks, we'll have to migrate to Gradle 8.4 first though (PR is underway)

what a coincidence, I just raised one, #519 :)

@fwcd fwcd added the ci-cd CI/CD-related (GitHub Actions) label Oct 11, 2023
@fwcd fwcd added the docker Related to the Docker image label Oct 22, 2023
fwcd and others added 2 commits January 14, 2024 22:54
@fwcd
Add Java 21 CI
54935f3
@chenrui333 @fwcd
dockerfile: update to use java 21
8271e5d 
Signed-off-by: Rui Chen <rui@chenrui.dev>
@fwcd
Copy link
Owner

fwcd commented Jan 14, 2024

With #537 (Gradle 8.5 and Kotlin 1.9.20), Java 21 should finally be supported. I've rebased this PR onto #538, which adds CI, unfortunately there is still a failing test to be addressed.

@chenrui333 chenrui333 closed this by deleting the head repository Jan 16, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ci-cd CI/CD-related (GitHub Actions) docker Related to the Docker image
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@chenrui333 @fwcd