-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
fix: typo in docs #10441
fix: typo in docs #10441
Conversation
| [Extensible Markup Language (XML)][xml] | `"xml"` | | | ||
| [JSON][json] | `"xpath_json"` | | | ||
| [MessagePack][msgpack] | `"xpath_msgpack"` | | | ||
| [Protocol buffers][protobuf] | `"xpath_protobuf"` | [see additional parameters](protocol-buffers-additiona-settings)| |
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.
@srebhan did you mean to include another doc for protobuf back when you added support?
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.
To be honest this should refer to the subsection below (line 18) but it seems I do have a typo there (protocol-buffers-additiona-settings
vs the correct protocol-buffers-additional-settings
)... :-(
@MyaLongmire it's a typo and therefore dead (see my comment to @reimda's comment). Could you maybe just fix the link instead of removing it? |
Co-authored-by: Sebastian Spaink <3441183+sspaink@users.noreply.github.com>
📦 Looks like new artifacts were built from this PR. Expand this list to get them here ! 🐯Artifact URLs |
(cherry picked from commit b2cfbb1)
Required for all PRs:
resolves #
Removes a link that leads to a 404 (I went through the file history and it never linked to anything that exsisted) and removes the section of the table as that was the only entry