Skip to content
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

Merged
merged 4 commits into from
Jan 25, 2022
Merged

fix: typo in docs #10441

merged 4 commits into from
Jan 25, 2022

Conversation

MyaLongmire
Copy link
Contributor

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

@telegraf-tiger telegraf-tiger bot added the fix pr to fix corresponding bug label Jan 13, 2022
| [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)|
Copy link
Contributor

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?

Copy link
Member

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)... :-(

@srebhan
Copy link
Member

srebhan commented Jan 25, 2022

@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?

@MyaLongmire MyaLongmire changed the title fix: remove dead link from docs fix: typo in docs Jan 25, 2022
Co-authored-by: Sebastian Spaink <3441183+sspaink@users.noreply.github.com>
@MyaLongmire MyaLongmire merged commit b2cfbb1 into master Jan 25, 2022
@MyaLongmire MyaLongmire deleted the protobuf-dead-link branch January 25, 2022 23:12
reimda pushed a commit that referenced this pull request Jan 27, 2022
(cherry picked from commit b2cfbb1)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix pr to fix corresponding bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants