-
Notifications
You must be signed in to change notification settings - Fork 25k
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
[DOCS] Add collapsible sections to 8.0 breaking changes [Part 4] #56356
[DOCS] Add collapsible sections to 8.0 breaking changes [Part 4] #56356
Conversation
Pinging @elastic/es-docs (>docs) |
*Impact* + | ||
Use the available constants (e.g. Fuzziness.ONE, Fuzziness.AUTO) or build your | ||
own instance using the above mentioned factory methods. Use only allowed | ||
`Fuzziness` values. |
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 Do you mind confirming that this accurately reflects your original guidance?
This is mostly a formatting change. I just want to ensure I don't unintentionally remove any context here.
Feel free to ignore the other changes in this PR unless you'd like to review them.
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.
Just a couple minor things.
Co-authored-by: debadair <[email protected]>
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
To go ahead and get the collapsible sections in, I've reverted most of the changes to the Java API changes section. I'll open a separate PR to get the impact sections of those changes approved. |
Adds collapsible sections to the 8.0 breaking changes.
To make reviews easier and avoid introducing errors, I've
tried to limit my changes to formatting, with the exception
of rewording heading as sentences.
I'm also splitting this effort over several PRs to make reviews
manageable.
Relates to #55629 and #53229
Preview
http://elasticsearch_56356.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/breaking-changes-8.0.html