-
Notifications
You must be signed in to change notification settings - Fork 25.2k
[docs] Fix various syntax and rendering errors #127062
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] Fix various syntax and rendering errors #127062
Conversation
Pinging @elastic/es-docs (Team:Docs) |
@@ -34,7 +34,7 @@ The following numeric types are supported: | |||
: A floating point number that is backed by a `long`, scaled by a fixed `double` scaling factor. | |||
|
|||
`unsigned_long` | |||
: An unsigned 64-bit integer with a minimum value of 0 and a maximum value of `2`^`64`^`-1`. | |||
: An unsigned 64-bit integer with a minimum value of 0 and a maximum value of `2^64 - 1`. |
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 think the problem with the superscripts is the pesky backticks
?
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.
So should we change this? If yes, can you provide a suggestion?
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.
"should" is strong, just pointing out that the superscript syntax won't work when code formatted, not high priority but if you're scripting these changes just a point to note for future :)
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.
In my opinion we should either format it like this (all inside code
like 2^64 - 1
) or with no code (just 264 - 1). Which do you prefer?
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.
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.
Spot checked a few pages and looks good to me 👍
💚 All backports created successfully
Questions ?Please refer to the Backport tool documentation |
* fix syntax and rendering errors * clean up * fix versions * more clean up * more fixes * more fixes * more fixes (cherry picked from commit 08552f1) # Conflicts: # docs/reference/elasticsearch/mapping-reference/dense-vector.md # docs/reference/elasticsearch/mapping-reference/semantic-text.md
* fix syntax and rendering errors * clean up * fix versions * more clean up * more fixes * more fixes * more fixes (cherry picked from commit 08552f1) # Conflicts: # docs/reference/elasticsearch/mapping-reference/dense-vector.md # docs/reference/elasticsearch/mapping-reference/semantic-text.md Co-authored-by: Colleen McGinnis <[email protected]>
Fixes various syntax and rendering errors that might include:
code
formatting in dropdown titles)deprecated:[8.15.0]
became[8.15.0]
, which doesn't give any information about what happened in 8.15.0)9.0.0
versions