8000 docs(external): introduce log namespace guide by pront · Pull Request #22985 · vectordotdev/vector · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

docs(external): introduce log namespace guide #22985

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

Merged
merged 13 commits into from
May 5, 2025

Conversation

pront
Copy link
Member
@pront pront commented May 1, 2025

Continuation of #22938

Renamed branch to generate website preview deployment

@pront pront requested a review from a team as a code owner May 1, 2025 18:45

This comment was marked as outdated.

This comment was marked as outdated.

@pront pront changed the title Pront website log namespace guide docs(external): introduce log namespace guide May 1, 2025
@pront pront added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label May 1, 2025
@datadog-vectordotdev
Copy link
datadog-vectordotdev bot commented May 1, 2025

Datadog Report

Branch report: pront-website-log-namespace-guide
Commit report: 0697fae
Test service: vector

✅ 0 Failed, 7 Passed, 0 Skipped, 25.44s Total Time

Copy link
github-actions bot commented May 1, 2025

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:
vector.dev preview

Copy link
github-actions bot commented May 1, 2025

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:
VRL Playground preview

@pront pront requested a review from jszwedko May 1, 2025 19:37
Copy link
Member
@jszwedko jszwedko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Thanks for generating the preview.

@pront
Copy link
Member Author
pront commented May 1, 2025

@joepeeples ready for a final review 🙏

Copy link
Contributor
@brett0000FF brett0000FF left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@pront pront added this pull request to the merge queue May 5, 2025
Merged via the queue into master with commit 61c418f May 5, 2025
72 checks passed
@pront pront deleted the pront-website-log-namespace-guide branch May 5, 2025 15:47
pront added a commit that referenced this pull request May 6, 2025
* feat(website): introduce log namespace guide

* fmt fixes

* simplify example, remove metadata

* Jesse's review points

* md lint fixes

* Apply suggestions from code review

Co-authored-by: Joe Peeples <joe.peeples@datadoghq.com>

* docs review

* grammar fix

* remove t1 from the first section and improve the decoder difference section

* improve second section

* remove remap from first section

* simplify second section even further

---------

Co-authored-by: Joe Peeples <joe.peeples@datadoghq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Changes in this PR do not need user-facing explanations in the release changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0