-
Notifications
You must be signed in to change notification settings - Fork 1.8k
feat(website): render input and output badges for all components #23204
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
Conversation
Your preview site for the VRL Playground will be ready in a few minutes, please allow time for it to build. Heres your preview link: |
Your preview site for the vector.dev will be ready in a few minutes, please allow time for it to build. Heres your preview link: |
Datadog ReportBranch report: ✅ 0 Failed, 7 Passed, 0 Skipped, 25.29s Total Time |
Your preview site for the VRL Playground will be ready in a few minutes, please allow time for it to build. Heres your preview link: |
Your preview site for the vector.dev will be ready in a few minutes, please allow time for it to build. Heres your preview link: |
Your preview site for the VRL Playground will be ready in a few minutes, please allow time for it to build. Heres your preview link: |
Your preview site for the vector.dev will be ready in a few minutes, please allow time for it to build. Heres your preview link: |
description: string | ||
relevant_when?: string | ||
tags?: #MetricTags | ||
name?: Name | ||
type?: #MetricType | ||
default_namespace?: string |
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.
We can think about improving this at a second phase.
</svg> | ||
</a> | ||
</span> | ||
<div class="mt-3 border rounded divide-y dark:border-gray-700 dark:divide-gray-700"> |
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.
Hmm, I don't remember deleting this but it renders the "?" href just fine.
{{ with $v.description }} | ||
<div class="mt-2 prose dark:prose-dark"> | ||
{{ . | markdownify }} | ||
</div> | ||
{{ end }} | ||
|
||
{{ range $k, $v := $v.type }} | ||
{{ with $v.examples }} | ||
<div class="mt-2"> | ||
<span> | ||
Examples | ||
</span> | ||
|
||
<div class="mt-1.5 flex flex-col space-y-1 text-sm"> | ||
{{ range . }} | ||
{{ template "code" . }} | ||
{{ end }} | ||
</div> | ||
</div> | ||
{{ end }} | ||
{{ end }} | ||
</div> | ||
{{ end }} | ||
</div> | ||
{{ end }} | ||
</div> | ||
{{ end }} | ||
{{ end }} |
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.
indentation is a little messed up here
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 will followup with a PR to add an new formatter for these. They are not simple HTML files. Need a Go template formatter.
Summary
Check the new badges and the new
Output Types
section in the Remap transform.Change Type
Is this a breaking change?
How did you test this PR?
Does this PR include user facing changes?
Notes
@vectordotdev/vector
to reach out to us regarding this PR.pre-push
hook, please see this template.cargo fmt --all
cargo clippy --workspace --all-targets -- -D warnings
cargo nextest run --workspace
(alternatively, you can runcargo test --all
)./scripts/check_changelog_fragments.sh
git merge origin master
andgit push
.Cargo.lock
), pleaserun
cargo vdev build licenses
to regenerate the license inventory and commit the changes (if any). More details here.References