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.

Sign up for GitHub

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

Jump to bottom

Add JDK21 Support #647

Merged
merged 11 commits into from
Nov 10, 2023
Merged

Add JDK21 Support #647

merged 11 commits into from
Nov 10, 2023

Conversation

wu-sheng
Copy link
Member

  • If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
  • Update the CHANGES log.

@wu-sheng wu-sheng added this to the 9.1.0 milestone Nov 10, 2023
@wu-sheng wu-sheng requested a review from kezhenxu94 November 10, 2023 02:26
kezhenxu94
kezhenxu94 previously approved these changes Nov 10, 2023
View reviewed changes
@wu-sheng
Copy link
Member Author

image

It should work now. Compiling the test in JDK21 seems to have issues. But anyway, that is not our concern.

@wu-sheng wu-sheng requested a review from kezhenxu94 November 10, 2023 03:42
@wu-sheng
Copy link
Member Author

@kezhenxu94 Could you approve again?

@wu-sheng wu-sheng merged commit c82287e into main Nov 10, 2023
182 checks passed
@wu-sheng wu-sheng deleted the jdk21 branch November 10, 2023 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kezhenxu94 kezhenxu94 kezhenxu94 approved these changes

Assignees
No one assigned
Labels
dependency testing
Projects
None yet
Milestone
9.1.0
Development

Successfully merging this pull request may close these issues.
2 participants
@wu-sheng @kezhenxu94