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

add java 22 to README #26495

Merged
merged 1 commit into from
Jun 20, 2024
Merged

add java 22 to README #26495

merged 1 commit into from
Jun 20, 2024

Conversation

mshima
Copy link
Member

@mshima mshima commented Jun 20, 2024


Please make sure the below checklist is followed for Pull Requests.

When you are still working on the PR, consider converting it to Draft (below reviewers) and adding skip-ci label, you can still see CI build result at your branch.

README.md Outdated
| 17 | 18 ||
| Java | Node | Status |
| -------- | ------- | ------ |
| 17/21/22 | 18/20 ||
Copy link
Contributor

Choose a reason for hiding this comment

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

Would it be accurate to say 17-21 and 18-20 or do you think it's better to specify the specific versions like you have here?

Copy link
Member Author

Choose a reason for hiding this comment

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

I think the old table not accurate, if looks like java 21 is tested with node 20 and java 17 with node 18.
We do have tests in place for java 17/21/22, not 18/19/20. Same for node, so specific versions seems better .

@DanielFran DanielFran merged commit f61d4b6 into main Jun 20, 2024
28 checks passed
@DanielFran DanielFran deleted the java22 branch June 20, 2024 23:27
@mraible mraible added this to the 8.6.0 milestone Jun 24, 2024
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.

None yet

3 participants