-
Notifications
You must be signed in to change notification settings - Fork 489
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
Add missing shared link syntax #5156
Conversation
@@ -135,7 +135,7 @@ Name | Type | Description | Default | Required | |||
|
|||
### output block | |||
|
|||
{{< docs/shared lookup="flow/reference/components/output-block-metrics.md" source="agent" >}} | |||
{{< docs/shared lookup="flow/reference/components/output-block-metrics.md" source="agent" version="<AGENT VERSION>" >}} |
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 see a few other occurrences of this in case we want to bundle them here.
$ grep -r "docs/shared lookup" | grep -v "VERSION"
docs/developer/writing-flow-component-documentation.md:{{< docs/shared lookup="flow/stability/beta.md" source="agent" >}}
docs/developer/writing-flow-component-documentation.md:{{< docs/shared lookup="flow/stability/experimental.md" source="agent" >}}
docs/sources/flow/reference/components/discovery.consulagent.md:{{< docs/shared lookup="flow/reference/components/tls-config-block.md" source="agent" >}}
docs/sources/flow/reference/components/otelcol.connector.servicegraph.md:{{< docs/shared lookup="flow/stability/experimental.md" source="agent" >}}
docs/sources/flow/reference/components/otelcol.connector.servicegraph.md:{{< docs/shared lookup="flow/reference/components/output-block-metrics.md" source="agent" >}}
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 can add them into the PR...
DONE
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, let's fix the rest of the occurrences too
PR Description
The shared link syntax was missing the
version="<AGENT VERSION>"
part. This PR adds it to the topic.Parent PR is: #5008
Which issue(s) this PR fixes
Notes to the Reviewer
PR Checklist