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

[chore]: bump developer guide java version #908

Merged

Conversation

IanMenendez
Copy link
Contributor

@IanMenendez IanMenendez commented Sep 14, 2024

Description

Bumped Java version in developer guide since it was outdated

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@yuye-aws
Copy link
Member

Can you also update the section title and description?

@IanMenendez
Copy link
Contributor Author

Missed that. Need some rest haha

@yuye-aws
Copy link
Member

Thank you. Since 2.x can still run on JDK 11 and 17, there is no need to backport this PR to 2.x.

Copy link
Member

@yuye-aws yuye-aws left a comment

Choose a reason for hiding this comment

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

Looks good to me.

Copy link
Member

@martin-gaievski martin-gaievski left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for tracking these things

@martin-gaievski
Copy link
Member

We can merge this one, CI is failing for some unrelated reason. @vibrantvarun can you please review?

@yuye-aws
Copy link
Member

Pinging @zane-neo and @zhichao-aws for review

Copy link
Member

@zhichao-aws zhichao-aws left a comment

Choose a reason for hiding this comment

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

lgtm

@zhichao-aws zhichao-aws merged commit 56b1649 into opensearch-project:main Sep 26, 2024
34 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants