-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Docs: HighLevelRestClient#ping #29070
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
Conversation
Add documentation for `HighLevelRestClient#ping`. Relates to elastic#28389
|
Pinging @elastic/es-core-infra |
| * // tag::example[] | ||
| * // end::example[] | ||
| * | ||
| * Where example is your tag name. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we need these comments any more. They were useful when I first built the extraction tool for education but I think something like what I have now is shorter and easier to read. Ideas? Suggestions?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
| * Code wrapped in {@code tag} and {@code end} tags is included in the docs. | ||
| */ | ||
| public class MainDocumentationIT extends ESRestHighLevelClientTestCase { | ||
| public class MiscellaneousDocumentationIT extends ESRestHighLevelClientTestCase { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I renamed this because I'd like to stick the other miscellaneous methods into this test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
++
cbuescher
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Add documentation for `HighLevelRestClient#ping`. Relates to #28389
Add documentation for
HighLevelRestClient#ping.Relates to #28389