-
Notifications
You must be signed in to change notification settings - Fork 24.9k
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
Remove included docs #50557
Remove included docs #50557
Conversation
In elastic#50499 we accidentally duplicated the docs for the `?local` parameter to the `GET _cat/nodes` API. This commit removes the duplicate docs.
Pinging @elastic/es-docs (>docs) |
Pinging @elastic/es-core-infra (:Core/Infra/Core) |
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 :)
deprecated:[7.6.0,This parameter does not cause this API to act locally. It will be removed in version 8.0.] | ||
If `true`, the request computes the list of selected nodes from the local | ||
cluster state. Defaults to `false`, which means the list of selected nodes is | ||
computed from the cluster state on the master node. In either case the |
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.
While we're at it: needs a ,
after "In either case" doesn't it? :)
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.
No, I think it's acceptable (and better) as it is.
In #50499 we accidentally duplicated the docs for the
?local
parameter to theGET _cat/nodes
API. This commit removes the duplicate docs.